Resolve "Internal Changes representation is not working as expected"

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

Closes #152

changed milestone to %v0.1.0 Codename Threadbare

added componentnucleus prioritycritical ~6539 typebug labels

assigned to @istmabauc

added 2 commits

• 550743e2 - Removed pushes to errChan from case: ticker, within the stateManager
• 7dc82f18 - Ticker stops now if a Change is confirmed

Compare with previous version

added 2 commits

• 2e560e9d - NewChange() starts a Goroutine acting as a Change's reciever for errors
• 07e89d50 - Rollback stops ticker and resets state of a Change

Compare with previous version

added 2 commits

• 6491005f - removed unnecessary code
• 954ce06a - Ticker will now start as soon as status committed is set and not at init

Compare with previous version

added 2 commits

• 1c1598f7 - moved the state checks into stateManager
• 49e292c8 - added a test case for nearly simultaneous calls of change.Commit()

Compare with previous version

added 1 commit

• 9e24fb52 - changed commit,confirm and state to use new channels for each request

Compare with previous version

added 1 commit

• ed9f5f8b - added individual error channel for each commit,confirm

Compare with previous version

added 2 commits

• b8cc1c9f - moved state into Change struct
• bdfd8419 - reverted some changes, removed testcase and added mutex

Compare with previous version

added 1 commit

• a8cc3b3e - debug test

Compare with previous version

added 1 commit

• 60ff9c3d - potential data race fix

Compare with previous version

added 1 commit

• fa127c36 - Instead of passing t.Name() via a channel, a copy is now created

Compare with previous version

added 1 commit

• dd4c1457 - refactoring and removed internal error channel

Compare with previous version

added 1 commit

• fa9a4f14 - added context which allows to interrupt

Compare with previous version

mentioned in issue #145 (closed)

approved this merge request

marked this merge request as ready

started a merge train

removed this merge request from the merge train