CMPUT301F23T01 / Sigma-Blue

Introduction to Software engineering project repository
BSD 2-Clause "Simplified" License
0 stars 1 forks source link

Yunyang/testing/UI testing1 #221

Closed Yunyang-Lu closed 10 months ago

Yunyang-Lu commented 10 months ago

First 3 UI tests for ViewListActivity is running, please check if its performance meets requirements.

lizard-52 commented 10 months ago

Both tests run but don't pass. The account was missing, but even after adding a new testing account they don't seem to work properly.

waridh commented 10 months ago

Both tests run but don't pass. The account was missing, but even after adding a new testing account they don't seem to work properly.

Do you think its because multiple people can't run the test at the same time?

lizard-52 commented 10 months ago

Both tests run but don't pass. The account was missing, but even after adding a new testing account they don't seem to work properly.

Do you think its because multiple people can't run the test at the same time?

I was actually on an old commit, looks good now

Yunyang-Lu commented 10 months ago

Both tests run but don't pass. The account was missing, but even after adding a new testing account they don't seem to work properly.

Do you think its because multiple people can't run the test at the same time?

It can be an issue when there is an error occurs, cause the test account do not get deleted. After that any test will not pass Current solution is login to firebase, and manually delete the account Which is bad, but I don't see any better solution can be done

waridh commented 10 months ago

Both tests run but don't pass. The account was missing, but even after adding a new testing account they don't seem to work properly.

Do you think its because multiple people can't run the test at the same time?

It can be an issue when there is an error occurs, cause the test account do not get deleted. After that any test will not pass Current solution is login to firebase, and manually delete the account Which is bad, but I don't see any better solution can be done

That's fine for now. Automation on that would require us opening the backend of the app somehow. It sounds possible, but not in the due date.

waridh commented 10 months ago

Both tests run but don't pass. The account was missing, but even after adding a new testing account they don't seem to work properly.

Do you think its because multiple people can't run the test at the same time?

It can be an issue when there is an error occurs, cause the test account do not get deleted. After that any test will not pass Current solution is login to firebase, and manually delete the account Which is bad, but I don't see any better solution can be done

That's fine for now. Automation on that would require us opening the backend of the app somehow. It sounds possible, but not in the due date.

Please clean up the code so that we could push this through. Clean code should be in our testing as well.

Yunyang-Lu commented 10 months ago

Refactoring done, part of commented out code are kept, can be useful afterwards