Skip to content
New issue

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

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

Already on GitHub? # to your account

various small tests ... #1786

Merged
merged 6 commits into from
Mar 21, 2024
Merged

various small tests ... #1786

merged 6 commits into from
Mar 21, 2024

Conversation

alaingdl
Copy link
Contributor

OK, this is a merging of various codes in GDL syntax for tests :

  • test_xyztick_get.pro tests [xyz]tick_get for PLOT & SURFACE, with various combinations of fields and ranges
    (@GillesDuvert that shows PR close #1781 (TICK_GET error) #1785 is not the definitive one)
  • test_operators.pro tests /=, *=, +=, -=. Might be redundant with test_indepth_basic_functions.pro
  • gdl_version.pro does manages the new version schema strip first char of version string for !GDL.RELEASE if it is "v" #1780 (for naming convention in i.e. bench out files)
  • test_foreach.pro merges some tests related to FOREACH (SF 273, 630 and test_bug_3288652.pro (removed outside)
  • bench_loops.pro

bench_loops_Linux_planck4

@alaingdl
Copy link
Contributor Author

@alaingdl alaingdl merged commit f92f857 into gnudatalanguage:master Mar 21, 2024
2 of 6 checks passed
@GillesDuvert
Copy link
Contributor

test_operators.pro tests /=, *=, +=, -=. Might be redundant with test_indepth_basic_functions.pro
yes, operators are there for "completion" but the real test that visit every possible line of code is indeed test_indepth_basic_functions.pro.
probably not worth the work to clean those duplicates.
and test_indepth_basic_functions.pro is really test_indepth_all_operators.pro.

@GillesDuvert
Copy link
Contributor

test_xyztick_get.pro has been invaluable in realizing AutoTick was wrong, thanks @alaingdl .

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants