Skip to content

Commit 08466ab

Browse files
authored
Merge pull request #183 from Unity-Lab-AI/develop
Removed playwrite, added page to build process
2 parents 7569f18 + ea3146c commit 08466ab

11 files changed

Lines changed: 33 additions & 1054 deletions

.github/workflows/deploy.yml

Lines changed: 0 additions & 70 deletions
Original file line numberDiff line numberDiff line change
@@ -171,73 +171,3 @@ jobs:
171171
echo ""
172172
echo "✨ Your site is now live with the latest updates!"
173173
echo "📦 Bundled, minified, and optimized for production"
174-
175-
# Job 4c: Run Playwright Tests
176-
playwright-tests:
177-
name: Run Playwright Tests
178-
needs: build
179-
runs-on: ubuntu-latest
180-
if: needs.build.outputs.build_status == 'success'
181-
182-
steps:
183-
- name: Checkout code
184-
uses: actions/checkout@v4
185-
with:
186-
ref: ${{ github.ref_name }}
187-
fetch-depth: 0
188-
189-
- name: Setup Node.js
190-
uses: actions/setup-node@v4
191-
with:
192-
node-version: '20'
193-
cache: 'npm'
194-
195-
- name: Install dependencies
196-
run: |
197-
echo "📦 Installing Playwright dependencies..."
198-
npm install
199-
200-
- name: Install Playwright browsers
201-
run: |
202-
echo "🌐 Installing Playwright browsers..."
203-
npx playwright install --with-deps
204-
205-
- name: Run Playwright tests
206-
run: |
207-
echo "🧪 Running Playwright tests..."
208-
npx playwright test
209-
env:
210-
CI: true
211-
212-
- name: Upload test results
213-
if: always()
214-
uses: actions/upload-artifact@v4
215-
with:
216-
name: playwright-report
217-
path: playwright-report/
218-
retention-days: 30
219-
220-
- name: Upload test results JSON
221-
if: always()
222-
uses: actions/upload-artifact@v4
223-
with:
224-
name: test-results
225-
path: test-results/
226-
retention-days: 30
227-
228-
- name: Report test success
229-
if: success()
230-
run: |
231-
echo "✅ PLAYWRIGHT TESTS PASSED"
232-
echo "================================"
233-
echo "All tests completed successfully"
234-
echo "================================"
235-
236-
- name: Report test failure
237-
if: failure()
238-
run: |
239-
echo "❌ PLAYWRIGHT TESTS FAILED"
240-
echo "================================"
241-
echo "Check test results for details"
242-
echo "================================"
243-
exit 1

.gitignore

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,6 @@ node_modules/
77
dist/
88

99
# Test results
10-
test-results/
11-
playwright-report/
12-
playwright/.cache/
1310
test-output*.txt
1411
test-output.log
1512
standalone-test-results.log

CLAUDE.md

Lines changed: 13 additions & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ The project features complete implementations of both libraries, a functional ma
3333
- HTML5, CSS3, JavaScript (vanilla)
3434
- Python 3.7+
3535
- Pollinations.AI API
36-
- Playwright for testing
3736
- GitHub Actions for CI/CD
3837

3938
---
@@ -96,10 +95,6 @@ sitetest0/
9695
│ ├── test_safety_and_reasoning.py # Safety/reasoning tests
9796
│ └── __init__.py # Package initialization
9897
99-
├── tests/ # Active Playwright tests
100-
│ ├── navigation.spec.js # Navigation tests (10 tests)
101-
│ └── browser-compatibility.spec.js # Browser compatibility tests
102-
10398
├── ai/ # AI Chat Section
10499
│ ├── index.html # AI landing page
105100
│ └── demo/ # Interactive demo (~90% complete, ~8,000 lines)
@@ -321,42 +316,6 @@ git commit -m "Docs: Update TODO list
321316

322317
## Testing Guidelines
323318

324-
### Playwright Tests
325-
326-
Located in `tests/` directory.
327-
328-
**Running Tests:**
329-
330-
```bash
331-
# Install dependencies
332-
npm install
333-
334-
# Run all tests
335-
npx playwright test
336-
337-
# Run specific test
338-
npx playwright test tests/navigation.spec.js
339-
340-
# Run with UI
341-
npx playwright test --ui
342-
343-
# Debug mode
344-
npx playwright test --debug
345-
```
346-
347-
### Test Organization
348-
349-
- **navigation.spec.js** - Navigation and page loading tests (10 tests)
350-
- **browser-compatibility.spec.js** - Browser compatibility & element loading tests across Chromium, Firefox, and WebKit
351-
- **README.md** - Test documentation and guidelines
352-
353-
### After Testing
354-
355-
1. Review test output
356-
2. Update `Docs/TEST_RESULTS.md` with findings
357-
3. Fix any failing tests before committing
358-
4. Document known issues or flaky tests
359-
360319
### Library Testing
361320

362321
**JavaScript:**
@@ -395,7 +354,6 @@ Both PolliLibJS and PolliLibPy follow the same modular structure:
395354
### Configuration Files
396355

397356
- **package.json** - NPM configuration
398-
- **playwright.config.js** - Playwright test configuration
399357
- **.github/workflows/** - CI/CD configuration
400358

401359
---
@@ -465,28 +423,25 @@ Both PolliLibJS and PolliLibPy follow the same modular structure:
465423
5. Commit changes in logical groups
466424
6. Update this CLAUDE.md if structure changes significantly
467425

468-
### Running a Full Test Suite
426+
### Running Library Tests
469427

470428
```bash
471-
# 1. Test website functionality
472-
npx playwright test
473-
474-
# 2. Test JavaScript library
429+
# 1. Test JavaScript library
475430
cd PolliLibJS
476431
node test-utils-demo.js
477432
cd ..
478433

479-
# 3. Test Python library
434+
# 2. Test Python library
480435
cd PolliLibPy
481436
python test_utils_demo.py
482437
cd ..
483438

484-
# 4. Update test results
439+
# 3. Update test results (if applicable)
485440
# Edit Docs/TEST_RESULTS.md with findings
486441

487-
# 5. Commit test results
442+
# 4. Commit test results
488443
git add Docs/TEST_RESULTS.md
489-
git commit -m "Test: Update test results after full suite run"
444+
git commit -m "Test: Update test results after library tests"
490445
```
491446

492447
---
@@ -501,7 +456,6 @@ git commit -m "Test: Update test results after full suite run"
501456
- SEO - Comprehensive implementation
502457
- Documentation - Well-documented
503458
- CSS/JS Minification - Automated deployment
504-
- Cross-browser testing - Firefox & WebKit 100% passing
505459

506460
### 🟢 Mostly Complete / Polishing
507461

@@ -516,7 +470,6 @@ git commit -m "Test: Update test results after full suite run"
516470
- Apps Gallery - In development (~70%)
517471
- ✅ Mini applications and utilities
518472
- ✅ Recent improvements (slideshow, navigation fixes)
519-
- Playwright tests - Working navigation and browser compatibility tests
520473
- Performance - Optimizations applied, ongoing improvements
521474

522475
### ❌ Not Started / External
@@ -552,7 +505,6 @@ git commit -m "Test: Update test results after full suite run"
552505
### External Resources
553506

554507
- [Pollinations.AI Documentation](https://github.com/pollinations/pollinations)
555-
- [Playwright Documentation](https://playwright.dev/)
556508
- [GitHub Actions Documentation](https://docs.github.com/en/actions)
557509

558510
---
@@ -572,9 +524,15 @@ This CLAUDE.md file should be updated when:
572524
---
573525

574526
**Last Updated:** 2025-11-22
575-
**Version:** 1.3.0
527+
**Version:** 1.4.0
576528
**Maintained by:** Unity AI Lab Team
577529

530+
**Change Log (v1.4.0):**
531+
- Removed all Playwright test references and documentation
532+
- Removed tests/ directory section from repository structure
533+
- Updated Testing Guidelines to focus on library testing only
534+
- Removed Playwright from Key Technologies and External Resources
535+
578536
**Change Log (v1.3.0):**
579537
- Removed references to non-existent `archived-tests/` directory
580538
- Added `/apps` gallery documentation (~70% complete)

README.md

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -145,19 +145,14 @@ sitetest0/
145145
├── apps/ # Mini apps gallery (~70% complete)
146146
│ └── ... (various utilities and mini apps)
147147
148-
├── tests/ # Playwright test suite
149-
│ ├── navigation.spec.js # Navigation tests (10 tests)
150-
│ ├── browser-compatibility.spec.js # Compatibility tests
151-
│ └── README.md # Test documentation
152-
153148
├── about/ # About page
154149
├── services/ # Services page
155150
├── projects/ # Projects page
156151
├── contact/ # Contact page
157152
├── index.html # Main landing page
158153
├── styles.css # Main stylesheet
159154
├── script.js # Main JavaScript
160-
├── CLAUDE.md # ⭐ AI assistant guide (v1.3.0)
155+
├── CLAUDE.md # ⭐ AI assistant guide (v1.4.0)
161156
└── README.md # This file
162157
```
163158

@@ -194,7 +189,6 @@ Both libraries are feature-complete and provide:
194189
- **Dark Gothic UI**: Immersive dark-themed interface
195190
- **Responsive Design**: Works on desktop and mobile devices
196191
- **Cache Busting**: Automated version control for assets
197-
- **Cross-browser Support**: Tested on Chromium, Firefox, and WebKit
198192

199193
## Authentication
200194

@@ -220,7 +214,7 @@ Changes tested here are promoted to the main Unity AI Lab website after validati
220214
## Documentation
221215

222216
### For Developers & AI Assistants
223-
- **⭐ AI Assistant Guide**: [CLAUDE.md](./CLAUDE.md) - Complete development guide (v1.3.0)
217+
- **⭐ AI Assistant Guide**: [CLAUDE.md](./CLAUDE.md) - Complete development guide (v1.4.0)
224218
- **⭐ Project Roadmap**: [Docs/TODO/TODO.md](./Docs/TODO/TODO.md) - Master TODO and project status
225219

226220
### API & Libraries

copy-assets.js

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,19 @@ const ASSETS_TO_COPY = [
2323
{ src: 'about/about.js', dest: 'about/about.js', type: 'file' },
2424
{ src: 'ai/demo/age-verification.js', dest: 'ai/demo/age-verification.js', type: 'file' },
2525
{ src: 'ai/demo/js/main.js', dest: 'ai/demo/js/main.js', type: 'file' },
26+
// Apps subdirectories (apps/index.html is handled by Vite)
27+
{ src: 'apps/helperInterfaceDemo', dest: 'apps/helperInterfaceDemo', type: 'dir' },
28+
{ src: 'apps/oldSiteProject', dest: 'apps/oldSiteProject', type: 'dir' },
29+
{ src: 'apps/personaDemo', dest: 'apps/personaDemo', type: 'dir' },
30+
{ src: 'apps/screensaverDemo', dest: 'apps/screensaverDemo', type: 'dir' },
31+
{ src: 'apps/slideshowDemo', dest: 'apps/slideshowDemo', type: 'dir' },
32+
{ src: 'apps/talkingWithUnity', dest: 'apps/talkingWithUnity', type: 'dir' },
33+
{ src: 'apps/textDemo', dest: 'apps/textDemo', type: 'dir' },
34+
{ src: 'apps/unityDemo', dest: 'apps/unityDemo', type: 'dir' },
35+
{ src: 'apps/shared-nav.html', dest: 'apps/shared-nav.html', type: 'file' },
36+
{ src: 'apps/shared-nav.js', dest: 'apps/shared-nav.js', type: 'file' },
37+
{ src: 'apps/shared-theme.css', dest: 'apps/shared-theme.css', type: 'file' },
38+
{ src: 'apps/update-apps.sh', dest: 'apps/update-apps.sh', type: 'file' },
2639
];
2740

2841
/**

package.json

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,6 @@
1010
"sitemap": "node generate-sitemap.js",
1111
"cache-bust": "node cache-bust.js",
1212
"copy-assets": "node copy-assets.js",
13-
"test": "playwright test",
14-
"test:chromium": "playwright test --project=chromium",
15-
"test:firefox": "playwright test --project=firefox",
16-
"test:webkit": "playwright test --project=webkit",
17-
"test:report": "playwright show-report",
1813
"minify": "npx terser script.js -c -m -o script.min.js && npx cleancss -o styles.min.css styles.css",
1914
"build:legacy": "npm run minify && ./update-version.sh"
2015
},
@@ -26,7 +21,6 @@
2621
"author": "",
2722
"license": "ISC",
2823
"devDependencies": {
29-
"@playwright/test": "^1.56.1",
3024
"clean-css-cli": "^5.6.3",
3125
"lighthouse": "^13.0.1",
3226
"terser": "^5.44.1",

playwright.config.js

Lines changed: 0 additions & 69 deletions
This file was deleted.

0 commit comments

Comments
 (0)