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

fre make create-makefile test #223

Open
5 tasks
singhd789 opened this issue Oct 28, 2024 · 0 comments
Open
5 tasks

fre make create-makefile test #223

singhd789 opened this issue Oct 28, 2024 · 0 comments
Assignees
@kiihne-noaa

Comments

@singhd789
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
fre make create-makefile needs an associated test.

Describe the solution you'd like
Tests could include:

  • existence of model yaml, platform yaml, and compile yaml
  • successful makefile creation (container - would be in tmp location, non-container)
  • test for proper link line defined if baremetal_linkerflags defined
  • test for creation of linkline.sh if container_addlibs defined
  • other failures.....
@singhd789 singhd789 moved this to Todo in fre make testing Oct 28, 2024
singhd789 added a commit that referenced this issue Nov 1, 2024
@singhd789
#223 Start fre make create_makefile test
71b474f
singhd789 added a commit that referenced this issue Nov 1, 2024
@singhd789
#223 Update test_create_makefile.py
179ce3f
singhd789 added a commit that referenced this issue Nov 4, 2024
@singhd789
#223 See where the test is being run
2f15391
singhd789 added a commit that referenced this issue Nov 4, 2024
@singhd789
#223 remove print statement
36fac3b
singhd789 added a commit that referenced this issue Nov 4, 2024
@singhd789
#223 See if combined-yaml relates to issue
a75bd25
singhd789 added a commit that referenced this issue Nov 4, 2024
@singhd789
#223 Remove os.chdir
9fb27cc 
- we dont want to change directories in tests
- changing directories seemed to be causing issues in `fre make` tool tests
singhd789 added a commit that referenced this issue Nov 4, 2024
@singhd789
#223 Move os.environ["HOME"] to be inside test
b814b99
singhd789 added a commit that referenced this issue Nov 4, 2024
@singhd789
#223 Remove chdir
8ba3aff
singhd789 added a commit that referenced this issue Nov 4, 2024
@singhd789
#223 Remove pytest skip
20ee9bf
singhd789 added a commit that referenced this issue Nov 4, 2024
@singhd789
#223 Remove unused import
f057f2f
singhd789 added a commit that referenced this issue Nov 4, 2024
@singhd789
#223 Apply pylint messages
6e3307b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kiihne-noaa kiihne-noaa

Labels
None yet
Projects
fre make testing
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
2 participants
@singhd789 @kiihne-noaa