Merge pull request #76 from Maple-Code-Crafters/mark-v1.0.0

Mark v1.0.0

Author: ravasconcelos

.browserslistrc

@@ -1,6 +1,6 @@
-Chrome >=79
-ChromeAndroid >=79
-Firefox >=70
-Edge >=79
-Safari >=14
-iOS >=14
+Chrome >=89
+ChromeAndroid >=89
+Firefox >=75
+Edge >=89
+Safari >=15
+iOS >=15

.eslintrc

@@ -1,4 +1,6 @@
 name: Tic-Tac-Toe CI
+env:
+  HUSKY: 0
 on: [push, pull_request]
 jobs:
   build-and-test:
@@ -7,10 +9,9 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version: '21'
-      - run: npm install -g vitest
-      - run: npm install @ionic/cli
-      - run: npm install rollup --save-dev
+          node-version: '22'
+          cache: npm
+      - run: npm i -g @ionic/cli
       - name: Install dependencies
         run: npm ci
       - name: Build

.github/workflows/tic-tac-toe-ci.yml

@@ -1,5 +1,5 @@
 echo "eslint running..."
-npx eslint --max-warnings=0 . --ext .tsx,.ts
+npx eslint . --max-warnings=0
 echo "tsc running..."
 npx tsc
 echo "prettier running..."

.husky/pre-commit

@@ -5,7 +5,6 @@ build/
 coverage/
 ios/
 resources/
-cypress/
 
 # Caches
 .eslintcache

.prettierignore

@@ -43,6 +43,8 @@
 
 `npm run test.unit`
 
+`npm run test.unit:debu <test file name>` to run only one test
+
 - [Cypress](https://docs.cypress.io/guides/overview/why-cypress)
 
 - Cypress needs the app/server running to work

README.md

@@ -2,7 +2,7 @@ apply plugin: 'com.android.application'
 
 android {
     namespace "io.ionic.starter"
-    compileSdkVersion rootProject.ext.compileSdkVersion
+    compileSdk rootProject.ext.compileSdkVersion
     defaultConfig {
         applicationId "io.ionic.starter"
         minSdkVersion rootProject.ext.minSdkVersion

android/app/build.gradle

@@ -15,6 +15,7 @@ dependencies {
     implementation project(':capacitor-preferences')
     implementation project(':capacitor-splash-screen')
     implementation project(':capacitor-status-bar')
+    implementation project(':sentry-capacitor')
 
 }
 

android/app/capacitor.build.gradle

@@ -7,8 +7,8 @@ buildscript {
         mavenCentral()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:8.3.0'
-        classpath 'com.google.gms:google-services:4.3.15'
+        classpath 'com.android.tools.build:gradle:8.3.2'
+        classpath 'com.google.gms:google-services:4.4.0'
 
         // NOTE: Do not place your application dependencies here; they belong
         // in the individual module build.gradle files

android/build.gradle

@@ -19,3 +19,6 @@ project(':capacitor-splash-screen').projectDir = new File('../node_modules/@capa
 
 include ':capacitor-status-bar'
 project(':capacitor-status-bar').projectDir = new File('../node_modules/@capacitor/status-bar/android')
+
+include ':sentry-capacitor'
+project(':sentry-capacitor').projectDir = new File('../node_modules/@sentry/capacitor/android')

android/capacitor.settings.gradle

@@ -1,14 +1,14 @@
 ext {
     minSdkVersion = 22
-    compileSdkVersion = 33
-    targetSdkVersion = 33
-    androidxActivityVersion = '1.7.0'
+    compileSdkVersion = 34
+    targetSdkVersion = 34
+    androidxActivityVersion = '1.8.0'
     androidxAppCompatVersion = '1.6.1'
     androidxCoordinatorLayoutVersion = '1.2.0'
-    androidxCoreVersion = '1.10.0'
-    androidxFragmentVersion = '1.5.6'
-    coreSplashScreenVersion = '1.0.0'
-    androidxWebkitVersion = '1.6.1'
+    androidxCoreVersion = '1.12.0'
+    androidxFragmentVersion = '1.6.2'
+    coreSplashScreenVersion = '1.0.1'
+    androidxWebkitVersion = '1.9.0'
     junitVersion = '4.13.2'
     androidxJunitVersion = '1.1.5'
     androidxEspressoCoreVersion = '3.5.1'

android/variables.gradle

@@ -0,0 +1,53 @@
+/// <reference types="cypress" />
+
+describe('Play page', () => {
+  describe('When visiting the root page then should have the New Game as header', () => {
+    it('Visits the app root url', () => {
+      cy.visit('/');
+      cy.contains('ion-title', 'New Game');
+      cy.get('ion-card').should('have.length', 1);
+      cy.get('ion-button').should('have.length', 1);
+      cy.get('ion-input').should('have.length', 2);
+    });
+  });
+
+  describe('Player name alert tests', () => {
+    it('When player1 name is not set then the alert shows the player1 message', () => {
+      cy.viewport('iphone-x');
+      cy.visit('/');
+      cy.get('[aria-label="reset"]').each(($el) => {
+        cy.wrap($el).click();
+      });
+      cy.get('ion-button').click();
+      cy.get('.alert-message').eq(0).contains('Please, enter player 1 name.');
+    });
+
+    it('When player1 name is set but player2 name is not set then the alert shows the player2 message', () => {
+      cy.viewport('iphone-x');
+      cy.visit('/');
+      cy.wait(1000);
+      //  clear player 2 name
+      cy.get('[aria-label="reset"]').each(($el, index) => {
+        if (index === 1) {
+          cy.wrap($el).click();
+        }
+      });
+      cy.get('ion-button').click();
+      cy.get('.alert-message').eq(0).contains('Please, enter player 2 name.');
+    });
+
+    it('When player names are set then there is no alert', () => {
+      cy.viewport('iphone-x');
+      cy.visit('/');
+      cy.wait(1000);
+      // click on two players button
+      cy.get('ion-segment-button').each(($el, index) => {
+        if (index === 1) {
+          cy.wrap($el).click();
+        }
+      });
+      cy.get('ion-button').click();
+      cy.get('.alert-message').should('have.length', 0);
+    });
+  });
+});

cypress/e2e/alert.cy.ts

@@ -34,4 +34,4 @@
 //       visit(originalFn: CommandOriginalFn, url: string, options: Partial<VisitOptions>): Chainable<Element>
 //     }
 //   }
-// }
+// }

cypress/e2e/pages/play.cy.ts

@@ -14,7 +14,7 @@
 // ***********************************************************
 
 // Import commands.js using ES2015 syntax:
-import './commands'
+import './commands';
 
 // Alternatively you can use CommonJS syntax:
-// require('./commands')
+// require('./commands')

cypress/e2e/test.cy.ts

@@ -0,0 +1,78 @@
+import eslintConfigPrettier from 'eslint-config-prettier';
+import jestDom from 'eslint-plugin-jest-dom';
+import reactRecommended from 'eslint-plugin-react/configs/recommended.js';
+import hooksPlugin from 'eslint-plugin-react-hooks';
+import simpleImportSort from 'eslint-plugin-simple-import-sort';
+import vitest from 'eslint-plugin-vitest';
+import globals from 'globals';
+import tseslint from 'typescript-eslint';
+
+import eslint from '@eslint/js';
+
+export default tseslint.config(
+  eslint.configs.recommended,
+  ...tseslint.configs.recommended,
+  {
+    ignores: ['ios/', 'android/', 'dist/', 'cypress.config.ts'],
+  },
+  {
+    settings: { react: { version: 'detect' } },
+    languageOptions: {
+      globals: globals.browser,
+    },
+  },
+  reactRecommended,
+  {
+    plugins: {
+      'react-hooks': hooksPlugin,
+    },
+    rules: hooksPlugin.configs.recommended.rules,
+  },
+  eslintConfigPrettier,
+  jestDom.configs['flat/recommended'],
+  {
+    files: ['**/*.test.ts?(x)'],
+    plugins: {
+      vitest,
+    },
+    rules: {
+      ...vitest.configs.recommended.rules,
+    },
+    languageOptions: {
+      globals: {
+        ...vitest.environments.env.globals,
+      },
+    },
+  },
+  {
+    plugins: {
+      'simple-import-sort': simpleImportSort,
+    },
+    rules: {
+      'no-console': 'warn',
+      'react/react-in-jsx-scope': 'off',
+      '@typescript-eslint/consistent-type-imports': 'warn',
+      'simple-import-sort/imports': [
+        'warn',
+        {
+          groups: [
+            // 1. `react` and packages starting with alphanumeric characters
+            ['^react$', '^\\w'],
+            // 2. vendor packages
+            ['^@'],
+            // 3. relative imports starting with "../"
+            ['^\\.\\./?$'],
+            // 4. relative imports from same folder "./"
+            ['^\\./?$'],
+            // 5. style imports
+            ['^.+\\.(css|scss)$'],
+            // 6. media imports
+            ['^.+\\.(gif|png|svg|jpg)$'],
+            // 7. side effect imports at the end
+            ['^\\u0000'],
+          ],
+        },
+      ],
+    },
+  },
+);

cypress/support/commands.ts

@@ -17,6 +17,7 @@ def capacitor_pods
   pod 'CapacitorPreferences', :path => '../../node_modules/@capacitor/preferences'
   pod 'CapacitorSplashScreen', :path => '../../node_modules/@capacitor/splash-screen'
   pod 'CapacitorStatusBar', :path => '../../node_modules/@capacitor/status-bar'
+  pod 'SentryCapacitor', :path => '../../node_modules/@sentry/capacitor'
 end
 
 target 'App' do