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

updated unit test to test that only published file is retrieved throu… #360

Merged

Conversation

tomshag
Copy link
Contributor

@tomshag tomshag commented Mar 18, 2024

…gh standard legacy getavailablefiles request

The code has been updated to filter FileTransferStatus retrieval based on FileTransferStatus.
This pull request adds a unit test that ensures that request for Published files will only retrieve Published files.

Description

The code has been updated to filter FileTransferStatus retrieval based on FileTransferStatus.
This pull request adds a unit test that ensures that request for Published files will only retrieve Published files.

Related Issue(s)

#251

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)
  • All tests run green

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

…gh standard legacy getavailablefiles request
@Andreass2
Copy link
Collaborator

The test itself looks good.

Format is failing, so should run a "dotnet format" command.

Copy link
Collaborator

@RagnarFatland RagnarFatland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good after whitespace fix.

@RagnarFatland RagnarFatland merged commit ae97970 into main Apr 2, 2024
1 check passed
@RagnarFatland RagnarFatland deleted the Feature/251-GetAvailableFiles_filestatus_unittests branch April 2, 2024 13:43
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