helpafamily icon indicating copy to clipboard operation
helpafamily copied to clipboard

🧪 ✨ Let's Get Code Coverage to 80% ✨

Open LowLifeArcade opened this issue 2 years ago • 6 comments

What should we change and why?

Thank you @idiglove 🙏 for adding test coverage to the Card component. We could use more! If you want to contribute to getting the code base test coverage to 80%, please join in 🦖

Check out #354 for an example.

LowLifeArcade avatar Mar 21 '22 22:03 LowLifeArcade

Hi,

My colleague Ana Rita and I are students at masters in software engineering at FEUP (https://paginas.fe.up.pt/~estudar/mesw/). As part of evaluating one of the curriculum units, we need to choose an open-source project and fix an issue. We would like to know if we could help with this issue?

FranciscoBastos avatar Mar 26 '22 11:03 FranciscoBastos

Hi @FranciscoBastos. Yes, absolutely! You and Ana can choose a component (or other parts of the codebase) and go from there with your testing! Try to keep your PRs limited to one piece of code or a single component. And have fun!

LowLifeArcade avatar Mar 26 '22 17:03 LowLifeArcade

Hi, @LowLifeArcade, could you assign me to the issue then? @RitaVeiga and I would like to take this issue!

FranciscoBastos avatar Apr 21 '22 08:04 FranciscoBastos

You got it @FranciscoBastos!

LowLifeArcade avatar Apr 22 '22 17:04 LowLifeArcade

Hi @LowLifeArcade me, and @RitaVeiga are getting the following error when running the tests, do you know how can we fix it? Captura de ecrã 2022-05-10, às 7 08 21 PM

FranciscoBastos avatar May 10 '22 18:05 FranciscoBastos

Hi, @LowLifeArcade @RitaVeiga and I have already been able to run the tests, and we have also written some test scripts. They are selenium test scripts that test the front-end of the application. They consist of E2E integration testing, and I think it was a reasonable effort on a first approach. We will keep updating the test suit so you can reach 80% of coverage!

FranciscoBastos avatar May 15 '22 12:05 FranciscoBastos