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

Add localfs unit tests #21

Open
hardillb opened this issue Jan 25, 2022 · 1 comment
Open

Add localfs unit tests #21

hardillb opened this issue Jan 25, 2022 · 1 comment
Assignees
Labels
size:S - 2 Sizing estimation point task A piece of work that isn't necessarily tied to a specific Epic or Story.
Milestone

Comments

@hardillb
Copy link
Contributor

hardillb commented Jan 25, 2022

Story

FlowFuse/flowfuse#11

Description

Add unit test framework to localfs driver

@knolleary knolleary moved this to Todo in 🛠 Development Jan 25, 2022
@sammachin sammachin added the task A piece of work that isn't necessarily tied to a specific Epic or Story. label Apr 4, 2022
@sammachin sammachin moved this to ✅ Ready in ☁️ Product Planning Apr 4, 2022
@sammachin sammachin moved this from ✅ Ready to 🧮 Estimate in ☁️ Product Planning Apr 4, 2022
@knolleary knolleary changed the title Add unit tests Add localfs unit tests Apr 6, 2022
@knolleary
Copy link
Member

We need to look at how feasible it is to run the code in this repo without the full forge stack in tow - otherwise we end up with complicated cross dependencies.

There is also the question of how much mocking we can get away with. I suspect this driver will be the easier one to have automated unit tests for compared to the others.

One approach would be to have a way to inject mocked nr-launcher that doesn't start NR, but provides a way to mock different behaviours (crashes, restarts etc)

The sizing estimate I'm applying to this task is based on getting the basic framework in place with a minimal set of tests - it is not intended to clear the full test debt backlog.

@knolleary knolleary added the size:S - 2 Sizing estimation point label Apr 6, 2022
@sammachin sammachin moved this from 🧮 Estimate to ✅ Ready in ☁️ Product Planning Apr 7, 2022
@sammachin sammachin moved this to Todo in 🛠 Development Jul 8, 2022
@sammachin sammachin added this to the 0.8 milestone Jul 8, 2022
@knolleary knolleary self-assigned this Jul 8, 2022
@sammachin sammachin modified the milestones: 0.8, 0.9 Aug 5, 2022
@sammachin sammachin modified the milestones: 0.9, 1.0 Sep 1, 2022
@sammachin sammachin modified the milestones: 0.10, 1.0 Sep 29, 2022
@knolleary knolleary modified the milestones: 1.0, 1.1 Oct 28, 2022
@knolleary knolleary moved this to ✅ Ready in ☁️ Product Planning Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S - 2 Sizing estimation point task A piece of work that isn't necessarily tied to a specific Epic or Story.
Projects
Status: No status
Development

No branches or pull requests

3 participants