Skip to content

test code improvement - remove gotException - #138

Open
dbulahov wants to merge 6 commits into
eclipse-daanse:mainfrom
dbulahov:main190826
Open

test code improvement - remove gotException#138
dbulahov wants to merge 6 commits into
eclipse-daanse:mainfrom
dbulahov:main190826

Conversation

@dbulahov

@dbulahov dbulahov commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Signed-off-by: dbulahov bulahovdenis@gmail.com

What does this PR do?

test improvement

  1. remove gotException construction
  2. migration boolean checks to isEqualTo, isSameAs, contains, `isInstanceOf
  3. add ValueSource for tests EmfAggMatchXmiRoundtripTest
  4. improvement
  5. remove empty afterEach beforeEach
  6. rename tests names to to descriptive names
  7. add descriptions for disabled tests
  8. structured error messages in assertOk

Signed-off-by: dbulahov <bulahovdenis@gmail.com>
…fAggMatchXmiRoundtripTest improvement

Signed-off-by: dbulahov <bulahovdenis@gmail.com>
Signed-off-by: dbulahov <bulahovdenis@gmail.com>
Signed-off-by: dbulahov <bulahovdenis@gmail.com>
Signed-off-by: dbulahov <bulahovdenis@gmail.com>
Signed-off-by: dbulahov <bulahovdenis@gmail.com>
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.

1 participant