Skip to content

test: expand tests for file transfer#1411

Merged
lusergit merged 1 commit intoedgehog-device-manager:mainfrom
ArnelaL:expand-tests
May 4, 2026
Merged

test: expand tests for file transfer#1411
lusergit merged 1 commit intoedgehog-device-manager:mainfrom
ArnelaL:expand-tests

Conversation

@ArnelaL
Copy link
Copy Markdown
Collaborator

@ArnelaL ArnelaL commented May 4, 2026

What this PR does / why we need it:

  • Expanded test coverage for the file transfer feature
  • Refactored and improved code to address bugs identified through the new tests

Additional documentation e.g. usage docs, diagrams, reviewer notes, etc.:


Thanks for sending a pull request! If this is your first time, here are some tips for you:
  1. You can take a look at our developer guide for an introduction on Edgehog development!
  2. Make sure to read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  3. If the PR is unfinished or you're actively working on it, mark it as draft

When fixing existing issues, use github's syntax to link your pull request to it

fixes #<issue number>

We also have a syntax to signal dependencies to other open pull requests

depends on #<pr number>
depends on https://github.com/...

In case of stacked PRs, you may add the PR number in the last commit's title instead:

gitGraph
    commit id: "Current master"
    branch feat1
    checkout feat1
    commit id: "feat: add something"
    commit id: "feat: add something else (#100)"
    branch feat2
    checkout feat2
    commit id: "refactor: do something"
    commit id: "fix: solve issue"
    commit id: "feat: add a feature (#101)"
    branch feat3
    checkout feat3
    commit id: "feat: feat without pr number"
Loading

@ArnelaL ArnelaL requested a review from OmarBrbutovic May 4, 2026 12:17
- Expanded test coverage for the file transfer feature
- Refactored and improved code to address bugs identified through
the new tests

Signed-off-by: ArnelaL <arnela.lisic@secomind.com>
@lusergit lusergit merged commit ce930ba into edgehog-device-manager:main May 4, 2026
19 checks passed
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