Fixing CI

Author: jlkalberer

.npmrc

@@ -1,3 +1,2 @@
 //npm.pkg.github.com/:_authToken=${NODE_AUTH_TOKEN}
 @brewskey:registry=https://npm.pkg.github.com
-always-auth=true

package-lock.json

@@ -21,7 +21,7 @@
     "test:e2e:expo:ui": "concurrently \"npm run web\" \"wait-on http://localhost:8081 && playwright test --ui\""
   },
   "dependencies": {
-    "@brewskey/js-api": "^2.0.5",
+    "@brewskey/js-api": "^2.0.6",
     "@expo/metro-runtime": "~6.1.2",
     "@frsty/slot-fill": "^0.2.4",
     "@hookform/error-message": "^2.0.1",

package.json

@@ -81,7 +81,6 @@ export const KegForm: React.FC<Props> = ({
     },
   });
 
-
   const kegType = form.watch('kegType');
 
   const selectedKegTypeMaxOunces = MAX_OUNCES_BY_KEG_TYPE[kegType] || 0;
@@ -137,7 +136,10 @@ export const KegForm: React.FC<Props> = ({
             {!shouldShowFloatedButton ? null : (
               <SubmitButton<FormFields>
                 allowSubmitWhenValid
-                buttonStyle={{ backgroundColor: COLORS.accent, marginBottom: 4 }}
+                buttonStyle={{
+                  backgroundColor: COLORS.accent,
+                  marginBottom: 4,
+                }}
                 onSubmit={onFloatKegForm}
                 title="Keg Floated"
               />

src/components/KegForm/index.tsx

@@ -79,7 +79,9 @@ export const LoginForm = ({ isInverse }: { isInverse: boolean }) => {
         <SectionContent paddedVertical>
           <SubmitButton<FormProps>
             allowSubmitWhenValid
-            buttonStyle={isInverse ? { backgroundColor: COLORS.secondary } : undefined}
+            buttonStyle={
+              isInverse ? { backgroundColor: COLORS.secondary } : undefined
+            }
             onSubmit={onSubmit}
             testID="login-submit-button"
             title="Log in"

src/components/LoginForm.tsx

@@ -29,8 +29,7 @@ const NewFlowSensorCustomScreen = withErrorBoundary(
     const _onFormSubmit = async (values: FlowSensorMutator): Promise<void> => {
       await createFlowSensor.mutateAsync(values);
       router.navigate({
-        pathname:
-          '/taps/[tapId]/keg/new',
+        pathname: '/taps/[tapId]/keg/new',
         params: {
           tapId: String(tapId),
           ...(returnTo ? { returnTo } : {}),

src/routes/(tabs)/(feed,stats,notifications,menu)/flow-sensor/custom.tsx

@@ -53,8 +53,7 @@ const NewFlowSensorScreen = withErrorBoundary(
         router.back();
       } else {
         router.navigate({
-          pathname:
-            '/taps/[tapId]/keg/new',
+          pathname: '/taps/[tapId]/keg/new',
           params: {
             tapId: String(tapId),
             ...(returnTo ? { returnTo } : {}),

src/routes/(tabs)/(feed,stats,notifications,menu)/flow-sensor/new.tsx

@@ -52,7 +52,9 @@ const EditTapPaymentsRouteContent: React.FC = () => {
   // All hooks must be called unconditionally before any early returns
   const form = useFormContext<PriceVariantMutator>();
   const isFormReady = form.formState != null;
-  const { formState: { errors, isSubmitting } } = form;
+  const {
+    formState: { errors, isSubmitting },
+  } = form;
 
   const addSnackBarMessage = useAddSnackBarMessage();
 
@@ -193,13 +195,13 @@ const EditTapPaymentsRouteContent: React.FC = () => {
           />
 
           <FormField<PriceVariantMutator, typeof TextInput>
-              component={TextInput}
-              defaultValue={(formValue != null ? formValue.ounces : 0).toFixed(1)}
-              keyboardType="numeric"
-              label="Ounces"
-              name="ounces"
-              required="Ounces is required"
-            />
+            component={TextInput}
+            defaultValue={(formValue != null ? formValue.ounces : 0).toFixed(1)}
+            keyboardType="numeric"
+            label="Ounces"
+            name="ounces"
+            required="Ounces is required"
+          />
           <FormField<PriceVariantMutator, typeof TextInput>
             component={TextInput}
             keyboardType="numeric"

src/routes/(tabs)/(feed,stats,notifications,menu)/taps/[tapId]/edit/payments.tsx

@@ -27,7 +27,9 @@ test('should navigate between bottom tabs', async ({ page }) => {
   const notificationsTab = page.getByTestId('tab-notifications');
   await expect(notificationsTab).toBeVisible();
   await notificationsTab.click();
-  await expect(page.getByTestId('button-delete-all-notifications')).toBeVisible();
+  await expect(
+    page.getByTestId('button-delete-all-notifications'),
+  ).toBeVisible();
 
   // Menu tab has testID - use that instead of text-based locator
   const menuTab = page.getByTestId('tab-menu');

tests/e2e/navigation/navigation.spec.ts

@@ -1,12 +1,12 @@
-import { test, expect } from '../../fixtures/test-fixtures';
 import { mockNewUserState } from '../../fixtures/entity-fixtures';
+import { expect, test } from '../../fixtures/test-fixtures';
 
 test('should display tap creation prompt', async ({ page, nuxPage }) => {
   // Set up explicit data: new user
   await mockNewUserState(page);
   await nuxPage.gotoTapStep();
 
-  await expect(page).toHaveURL(/.*nux.*tap/i);
+  await expect(page).toHaveURL('/tap');
   // Tap screen has testID - use that instead of text-based locator
   await expect(page.getByTestId('nux-tap-content')).toBeVisible();
   await expect(page.getByTestId('nux-tap-description')).toBeVisible();
@@ -39,8 +39,7 @@ test('should navigate to taps/new when Next is clicked with deviceId', async ({
   page,
   nuxPage,
 }) => {
-  const { mockDeviceWithTaps } =
-    await import('../../fixtures/entity-fixtures');
+  const { mockDeviceWithTaps } = await import('../../fixtures/entity-fixtures');
   await mockNewUserState(page);
   const { device } = await mockDeviceWithTaps(page, 0);