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

getting started/sbt: compile -> Test/compile #162

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

numero-744
Copy link
Collaborator

Tests are being moved to specific folder so this line should be updated.

Note: this is an untested quickfix from GH web interface

Tests are being moved to specific folder so this line should be updated.
@numero-744
Copy link
Collaborator Author

Hmm this PR is good enough: if we move simulations to tb we have to do Test/runMain so it would need more explanations 🤔

@Readon
Copy link
Collaborator

Readon commented Dec 8, 2022

I think runMain and test could not be mixed. runMain is used to generate the RTL for the design, while test is used to test the design.
so either use sbt runMain xxxx.xxxx or sbt testOnly xxxx.xxxx for a simple case.

@numero-744
Copy link
Collaborator Author

The current way to do things is that a simulation test is defined extending App in main so currently a user should do runMain to execute this simulation test. So runMain is currently not only to generate the RTL (IMO it should be only for generation once SpinalTestBench is released).

So waiting for SpinalTestBench before adding a folder for tests is better?

@Dolu1990
Copy link
Member

Dolu1990 commented Dec 8, 2022

I think runMain and test could not be mixed. runMain is used to generate the RTL for the design, while test is used to test the design.
so either use sbt runMain xxxx.xxxx or sbt testOnly xxxx.xxxx for a simple case.

I would say, the main distinction between main / Test is that main will be contained in the release jar, while test will not be included.
The scala test framework is something else (coupled to Test)

@numero-744
Copy link
Collaborator Author

numero-744 commented Dec 9, 2022 via email

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