Simplify climbing screen on the scouting app
This patch gets rid of the secondary screen that popped up when
selecting "Attempted to Climb". It was confusing a few people. Now all
options are on the very first screen. Also, I decided to get rid of
all the `set*()` functions. It was simpler to use a two-way connection
for the radio buttons. I also got rid of the "proper" concept. I'm
still not sure what that was all about.
Signed-off-by: Philipp Schrader <philipp.schrader@gmail.com>
Change-Id: Iaa9c2dda6bb5af2aa14dae50dbe20bcc96ef1bbe
diff --git a/scouting/scouting_test.ts b/scouting/scouting_test.ts
index 2ffa136..2cd3e43 100644
--- a/scouting/scouting_test.ts
+++ b/scouting/scouting_test.ts
@@ -56,6 +56,7 @@
await element(by.buttonText('Next')).click();
expect(await getHeadingText()).toEqual('Climb');
+ await element(by.id('high')).click();
await element(by.buttonText('Next')).click();
expect(await getHeadingText()).toEqual('Defense');
@@ -79,7 +80,7 @@
await expectNthReviewFieldToBe('Missed Shots', 1, '0');
// Validate Climb.
- await expectReviewFieldToBe('Attempted to Climb', 'No');
+ await expectReviewFieldToBe('Level', 'High');
// Validate Defense.
await expectReviewFieldToBe('Defense Played On Rating', '0');