8000 Test code cleaning by hmiomand · Pull Request #383 · preesm/preesm · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Test code cleaning #383

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
Merged
merged 104 commits into from
Jun 24, 2024
Merged

Test code cleaning #383

merged 104 commits into from
Jun 24, 2024

Conversation

hmiomand
Copy link
Contributor
@hmiomand hmiomand commented Jun 16, 2024 •
edited
Loading

In this PR:

  • Updated CI Actions
  • Updated CI to make use of dependency cache
  • Fixed a copy/pasting issue with a delay with dependencies in the selection
  • Fixed a .pi/.diagram desync issue due to undo action (ctrl+z)
  • Fixed an issue with timing resolution during scheduling if the scenario timing expression contained parameter names (config input port) differing from graph parameter name. In case of name match, the value of the same-named graph parameter would be used instead.
  • Lots of code cleaning
  • Added some integration tests
  • Renamed malleable parameter to moldable
  • Fixed an issue where special actors could be set up with improper number of input/output data port (e.g. 2 inputs on a fork), causing the graph to be invalid and become unopenable and uneditable
  • Updated some project dependencies
  • Dependencies can now be created by clicking on any part of a data interface, instead of specifically its name
  • Fixed an issue where the diagram view itself could be deleted
  • SrDAG transformation now properly renames the graph instead of just the filename
  • Preesm-apps integration tests now run in parallel in Eclipse (for now)
  • Fixed a renaming issue when copy/pasting interfaces where the port contained in the interface would not be renamed
  • Fixed a copy/pasting issue when a ConfigOutputPort was connected to a fifo

hmiomand added 30 commits April 16, 2024 21:08
@hmiomand
JEPCustomFunctionTest: replaced set of tests by Parameterized test
1bc056a
@hmiomand
JavaExpressionParserTest: Replaced set of externally generated tests …
2ba179c 
…by runtime-generated tests with TestFactory
@hmiomand
PiGraphLookupTest: Separated tests
359b50a
@hmiomand
Moving test resources to fix failing CI
e6b0df8
@hmiomand
TopologyTest: Refacto to make use of assertThrows
8555d19
@hmiomand
Removed code smell
ec9b652
@hmiomand
Github Actions: updated actions to address upcoming node16 deprecation
eebd651
@hmiomand
PiSDFSubgraphBuilderTest: small refacto
5003804
@hmiomand
Merged and remove duplicate tests in GraphStructureHelper
59123aa
@hmiomand
Renamed org.ietr.preesm.mathematicalModels.test package
5b6430c
@hmiomand
Cleanup in org.ietr.preesm.schedule.test
abebc57
@hmiomand
Cleanup in TransformerTest
2b77619
@hmiomand
Removed specific exception handling in JEPCustomFunctionTest
3374110
@hmiomand
Cleanup in deadlock tests
b9967b8
@hmiomand
Removed barely used and outdated JMathArray dependency for the alread…
f5e8cf6 
…y used commons-math3
@hmiomand
Code cleaning in org.preesm.algorithm.model.sdf
2f5161e
@hmiomand
code cleaning in PiGraphFiringBalancerTest
a6730df
@hmiomand
Fix unit test not failing
eaf8971
@hmiomand
Fixed matrix rank result
f10cee7
@hmiomand
Code cleaning in GraphStructureHelperTest
3382081
@hmiomand
Fix typo in test name ReflectionUtilTest
798e8df
@hmiomand
Updated dependencies and removed unused ones
b40c62b
@hmiomand
Updated JDK to 17.0.10
d4b0860
@hmiomand
Updated dependencies in pom.xml
e51aaf7
@hmiomand
Removed a checkstyle rule that started to be applied after a checksty…
2416b67 
…le plugin update.

Fixed checkstyle violation that started to be flagged after a checkstyle plugin update.
@hmiomand
Removed code smell in org.preesm.algorithm.synthesis.schedule.algos
47b1773
< 8000 div class="AvatarStack-body" > @hmiomand
Enforce immutability in RefinementChecker accepted header file extens…
93c551a 
…ions
@hmiomand
Modified regex in RefinementChecker.isHlsStreamTemplated and added te…
4c08206 
…st cases
@hmiomand
Code cleaning in XMLStatsExporter
2bc25e2
@hmiomand
Code cleaning in AbstractPiSDFObjectChecker
ab89740
hmiomand added 27 commits May 10, 2024 22:57
@hmiomand
Improved copy/paste RCPTT test
6bfe422
@hmiomand
Added test for Activity Tasks
6c808ac
@hmiomand
Added test for BRV Exporter Task
5698530
@hmiomand
[UI] Refactoring of Add Features
697a18e
@hmiomand
Code cleaning
ca4b7f6
@hmiomand
Updated graph manipulation rcptt test
96f09a1
@hmiomand
SRDAG transformation now properly append to "_srdag" suffix to graph …
86b1e04 
…filename and to graph actual name
@hmiomand
Code cleaning
dea442f
@hmiomand
Updated graph manipulation rcptt test
b5ea621
@hmiomand
Code Cleaning
a3d5c4c
@hmiomand
Added RCPTT test for parameter dependency detection in interface
82f5468
@hmiomand
Replaced deprecated calls to AbstractUIPlugin
8419e3a
@hmiomand
Code Cleaning
8379024
@hmiomand
Renamed variables to match code style
4a25bf2
@hmiomand
Code Cleaning
ff76423
@hmiomand
Fix an issue where timing expression resolution during scheduling cou…
6fc73eb 
…ld be incorrect or throw exception if the name of a config input port of an actor was not the same as any graph parameter.
@hmiomand
Code Cleaning
c7b5900
@hmiomand
Code Cleaning
de8417e
@hmiomand
Code Cleaning
851dad3
@hmiomand
Code cleaning
4d9af13
@hmiomand
Small code cleaning in spider2 codegen
76f6077
@hmiomand
Added test for AutoDelayTask
8000 cabff32
@hmiomand
Merge branch 'develop' into test_code_cleaning
ffc3ab7 
Merge develop into test_code_cleaning
@hmiomand
Code cleaning
d47205e
@hmiomand
Added use of dependency cache
5a3ca00
@hmiomand
Fixed CI dependency cache creation
ed074ed
@hmiomand
Fixed changelog upload action running when not required
e0ffd2a
@hmiomand hmiomand merged commit 4d0167e into develop Jun 24, 2024
2 of 3 checks passed
@hmiomand hmiomand deleted the test_code_cleaning branch June 27, 2024 13:36
@hmiomand hmiomand mentioned this pull request Nov 20, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hmiomand
0