Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature Fix Unit Tests #63

Merged
merged 113 commits into from
Nov 8, 2022
Merged

Feature Fix Unit Tests #63

merged 113 commits into from
Nov 8, 2022

Conversation

pLeminoq
Copy link
Contributor

@pLeminoq pLeminoq commented Jul 5, 2022 •
edited by DivineThreepwood
Loading

@DivineThreepwood DivineThreepwood self-requested a review July 5, 2022 17:23
DivineThreepwood and others added 18 commits July 5, 2022 19:37
@DivineThreepwood
improve exception handling
00d0897
@DivineThreepwood
Increase testRegisteringManyDevices timeout
e754626
@DivineThreepwood
update jul
5b5818e
@DivineThreepwood
increase FINISHED_ACTION_REMOVAL_TIMEOUT for tests to 30 sec
fc59f74
@DivineThreepwood
Extend timeouts of testReinit()
248f304
@DivineThreepwood
link latest jul version
14129ee
@pLeminoq
add test dependencies to global gradle.kts
ad93653
@pLeminoq
Merge branch 'feature/tests' of https://github.com/openbase/bco into …
34a6062 
…feature/tests
@DivineThreepwood
Fix stay logged in test and port it to kotlin
867819e
@DivineThreepwood
fix typo
8aeae74
@DivineThreepwood
Improve UnitRegistry registry error logging to identify location loop…
16c561d 
… bug.
@pLeminoq
add test for location loop debugging
8af5095
@DivineThreepwood
fix location unit id list self reference.
88a3b07
@pLeminoq
add unit test for issue #65
8e85b21
@pLeminoq
remove test used for reproducing #65
393d8d4
@pLeminoq
optimize imports
c53cb27
@pLeminoq
allow unit groups to be members of other unit groups if their types m…
540386c 
…atch
@pLeminoq
add a unit test for #66
e29eb15
@DivineThreepwood DivineThreepwood added prebuild docker image Auto build a docker image and deploy it as a review version. and removed force reports This flag forces the CI Actions to report gradle logs even when successful. labels Oct 8, 2022
github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 27, 2022
View reviewed changes
module/dal/lib/src/test/java/org/openbase/bco/dal/lib/action/ActionDescriptionProcessorTest.kt
)
Assertions.assertEquals(
"SceneMoodLight",
ActionDescriptionProcessor.getUnitChainSuffixForNonReplaceableAction(sceneMoodLightActionDescription.build()),

Check warning

Code scanning / detekt

Line detected that is longer than the defined maximum line length in the code style.

Line detected that is longer than the defined maximum line length in the code style.
DivineThreepwood and others added 16 commits October 27, 2022 20:36
@DivineThreepwood
Merge remote-tracking branch 'origin/feature/tests' into bugfix/#72_f…
8ca3288 
…ix_index_out_of_bounds_in_influx_app
@pLeminoq
Merge pull request #76 from openbase/bugfix/#72_fix_index_out_of_boun…
3c45dde 
…ds_in_influx_app

fix index out of bounds in InfluxDbconnectorApp that might causes not…
@DivineThreepwood
update jul ref
9a19026
@DivineThreepwood
Merge remote-tracking branch 'origin/bugfix/#72_fix_index_out_of_boun…
e8254e8 
…ds_in_influx_app' into feature/tests
@DivineThreepwood
remove mergeFrom from ActionImpl after action execution. It was anywa…
36133fe 
…y never used.
@pLeminoq
replace mergeFrom calls with new method removing repeated fiels
4895d3d
@DivineThreepwood
update jul
106378e
@DivineThreepwood
Merge branch 'bugfix/#77_review_protobuf_merge_from_usage_to_avoid_li…
47d8e18 
…st_entry_dublication' of https://github.com/openbase/bco into bugfix/#77_review_protobuf_merge_from_usage_to_avoid_list_entry_dublication
@DivineThreepwood
update jul
b2ceab2
@DivineThreepwood
Update module/dal/control/src/main/java/org/openbase/bco/dal/control/…
d75f0fa 
…layer/unit/AbstractUnitController.java
@DivineThreepwood
update jul
bd8beda
@pLeminoq
Merge pull request #78 from openbase/bugfix/#77_review_protobuf_merge…
5a4a850 
…_from_usage_to_avoid_list_entry_dublication

Bugfix/#77 review protobuf merge from usage to avoid list entry duplication
@DivineThreepwood
link latest jul lib
2c34aad
@pLeminoq
re-add use of mergeFrom in MockRegistry
972f2c6
@pLeminoq
Merge branch 'feature/tests' of https://github.com/openbase/bco into …
e4b9fc8 
…feature/tests
@DivineThreepwood
test code improvements
7c55cda
@pLeminoq pLeminoq mentioned this pull request Nov 8, 2022
Merged
@DivineThreepwood DivineThreepwood merged commit c25a690 into dev Nov 8, 2022
@DivineThreepwood DivineThreepwood deleted the feature/tests branch November 8, 2022 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DivineThreepwood DivineThreepwood Awaiting requested review from DivineThreepwood

Assignees
No one assigned
Labels
prebuild docker image Auto build a docker image and deploy it as a review version.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@pLeminoq @DivineThreepwood