Skip to content

test: component: change comp_set_state test cases - #134

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
bkokoszx:test_component_set_state
Jul 20, 2018
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
bkokoszx:test_component_set_state

Conversation

@bkokoszx

Copy link
Copy Markdown
Collaborator

Update comp_set_state test cases due to refinement the component's
state machine in commit 5198517.

Signed-off-by: Bartosz Kokoszko bartoszx.kokoszko@linux.intel.com

Update comp_set_state test cases due to refinement the component's
state machine in commit 5198517.

Signed-off-by: Bartosz Kokoszko <bartoszx.kokoszko@linux.intel.com>
@lgirdwood

Copy link
Copy Markdown
Member

@bkokoszx Good catch, @zhigang-wu we need to use your state diagram in the docs, it should be generated using rst (as part of the code comments) so that others can align (including cmocka).

@lgirdwood
lgirdwood merged commit 4b62c25 into thesofproject:master Jul 20, 2018
@jajanusz

Copy link
Copy Markdown
Contributor

#153
test names with prefix comp_* should be test_audio_component_*

@lgirdwood

Copy link
Copy Markdown
Member

@bkokoszx please fix incrementally

@bkokoszx
bkokoszx deleted the test_component_set_state branch November 22, 2018 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants