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

Add unit tests for AM server #1036

Closed
andreleblanc11 opened this issue May 2, 2024 · 1 comment
Closed

Add unit tests for AM server #1036

andreleblanc11 opened this issue May 2, 2024 · 1 comment
Assignees
@andreleblanc11
Labels
coverage unit or flow tests do not notice changes here Priority 3 - Important worrisome impact... should study carefully sundew-migration issues with moving legacy configurations to Sarracenia v3only Only affects v3 branches.

Comments

@andreleblanc11
Copy link
Member

Placeholder for developing unit tests for the AM server
@andreleblanc11 andreleblanc11 added Priority 3 - Important worrisome impact... should study carefully sundew-migration issues with moving legacy configurations to Sarracenia v3only Only affects v3 branches. coverage unit or flow tests do not notice changes here labels May 2, 2024
@andreleblanc11 andreleblanc11 self-assigned this May 2, 2024
andreleblanc11 added a commit that referenced this issue May 7, 2024
@andreleblanc11
#1036 First attempt at adding unit test for one use case. Got it to w…
cea43f2 
…ork. Other use cases are commented for now
andreleblanc11 added a commit that referenced this issue May 7, 2024
@andreleblanc11
#1036 Add two more unit tests - Regular CACN and erronous CACN
f657742
andreleblanc11 added a commit that referenced this issue May 13, 2024
@andreleblanc11
#1036 Add rest of unit tests
2daa967
andreleblanc11 added a commit that referenced this issue May 13, 2024
@andreleblanc11
#1036 Correct code based off of unit test findings - Incorrect statio…
57f1f57 
…n names and longer bulletin headers
@andreleblanc11
Copy link
Member Author

andreleblanc11 commented May 14, 2024 •
edited
Loading

We'll also need to add tests for bulletin.py and the renamer flowcb/rename/raw2bulletin.py to get complete coverage.

@andreleblanc11 andreleblanc11 mentioned this issue May 14, 2024
Merged
petersilva pushed a commit that referenced this issue May 18, 2024
@andreleblanc11 @petersilva
#1036 First attempt at adding unit test for one use case. Got it to w…
3b71c58 
…ork. Other use cases are commented for now
petersilva pushed a commit that referenced this issue May 18, 2024
@andreleblanc11 @petersilva
#1036 Add two more unit tests - Regular CACN and erronous CACN
93e157d
petersilva pushed a commit that referenced this issue May 18, 2024
@andreleblanc11 @petersilva
#1036 Add rest of unit tests
b6c3daa
petersilva pushed a commit that referenced this issue May 18, 2024
@andreleblanc11 @petersilva
#1036 Correct code based off of unit test findings - Incorrect statio…
9716b30 
…n names and longer bulletin headers
andreleblanc11 added a commit that referenced this issue May 27, 2024
@andreleblanc11
#1036 Add more AM unit tests for latest bug findings
4d6af5f
petersilva added a commit that referenced this issue May 27, 2024
@petersilva
Merge pull request #1068 from MetPX/issue1036
2df0acf 
#1036 Add more AM unit tests for latest bug findings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andreleblanc11 andreleblanc11

Labels
coverage unit or flow tests do not notice changes here Priority 3 - Important worrisome impact... should study carefully sundew-migration issues with moving legacy configurations to Sarracenia v3only Only affects v3 branches.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andreleblanc11