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.

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

Rework test functional #2179

Merged
merged 3 commits into from
May 7, 2024
Merged

Conversation

ftoromanoff
Copy link
Contributor

@ftoromanoff ftoromanoff commented Aug 31, 2023

The purpose of this PR is do rewrite the functionnal test to limit the failure during githubaction linked to unclear reasons.

Changes:

  • fix some issue on hook_functional.js:
    • few minor corrections
    • after initialization save position only once and not after each test
    • gestion of initialization layer timing out
      • better message when encoutering an error
      • and real gestion of timeouterror (initialization of view > 50 000ms => stop camera movement)
  • reorganization of the functional tests for a better read (moved to Reorg functional test files #2324 )
    • creation of folder per functionalities tested

@ftoromanoff ftoromanoff marked this pull request as draft August 31, 2023 15:49
@ftoromanoff ftoromanoff requested a review from mgermerie August 31, 2023 15:49
@ftoromanoff ftoromanoff force-pushed the reworkTestFunctional branch 4 times, most recently from 2472956 to 2328132 Compare August 31, 2023 16:19
@ftoromanoff ftoromanoff marked this pull request as ready for review September 4, 2023 16:07
@ftoromanoff ftoromanoff force-pushed the reworkTestFunctional branch 3 times, most recently from 6a194b7 to c3b1d5d Compare September 15, 2023 14:11
@jailln jailln self-requested a review February 28, 2024 11:05
test/functional/hooks_functional.js Outdated Show resolved Hide resolved
test/functional/hooks_functional.js Outdated Show resolved Hide resolved
@@ -239,7 +263,7 @@ exports.mochaHooks = {
);
view.controls.lookAtCoordinate(init, false);
view.notifyChange();
} else if (view.isPlanarView) {
} else if (view && view.isPlanarView) {
// TODO: make the controls accessible from PlanarView before doing
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same remark here regarding the controls

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

View.controls is not yet refactored between all the different type of Views.
Thes part will need to be done before we can really cahnge the TODOs in here.

test/functional/examples.js Outdated Show resolved Hide resolved
@jailln jailln self-assigned this Feb 28, 2024
@ftoromanoff ftoromanoff force-pushed the reworkTestFunctional branch 2 times, most recently from d04f28d to 32f7812 Compare May 2, 2024 15:41
@ftoromanoff ftoromanoff requested a review from Desplandis May 3, 2024 12:52
@ftoromanoff ftoromanoff force-pushed the reworkTestFunctional branch from 32f7812 to b734c9a Compare May 6, 2024 09:18
test/hooks_functional.js Outdated Show resolved Hide resolved
test/hooks_functional.js Outdated Show resolved Hide resolved
test/hooks_functional.js Outdated Show resolved Hide resolved
Copy link
Contributor

@Desplandis Desplandis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Validated.

@ftoromanoff ftoromanoff force-pushed the reworkTestFunctional branch from 4970f66 to 9c101a9 Compare May 7, 2024 10:08
@ftoromanoff ftoromanoff merged commit 14891fd into iTowns:master May 7, 2024
9 checks passed
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