Merge pull request #567 from elsassph/fix/playwright

Fix playwright tests

Author: michielvandergeest

README.md

@@ -31,6 +31,28 @@ The Lightning is an open source project. If you want to contribute to it, please
 Before you submit your PR, make sure you install the projects dependencies, as this will activate automatic
 linting and code formatting in a Git commit hook.
 
+## Testing
+
+Make sure changes and new code are covered with unit tests or/and integration tests.
+
+### Unit tests (vitest)
+
+Test files are under `src/` with a `.test.mjs` or `.test.mts` extension.
+
+```
+npm run test
+```
+
+### Integration tests (Playwright)
+
+Test cases are under `tests/` and use either the ES5 build output or directly point to `.mjs` files (TypeScript won't run in the browser).
+
+```
+npx playwright install
+npm run build
+npm run playwright:interactive
+```
+
 ## Changelog
 
 Checkout the changelog [here](./CHANGELOG.md).

tests/event-emitter/test.event-emitter.js

@@ -17,8 +17,6 @@
  * limitations under the License.
  */
 
-import lng from '../../src'
-
 describe('Event Emitter', function() {
     let TestObj;
 

tests/keyhandler/test.keyhandleLong.js

@@ -17,8 +17,6 @@
  * limitations under the License.
  */
 
-import lng from '../../src'
-
 describe('Longpress handling', function() {
     this.timeout(0);
     let app;

tests/keyhandler/test.keyhandler.js

@@ -17,8 +17,6 @@
  * limitations under the License.
  */
 
-import lng from '../../src'
-
 describe('Key handling', function() {
     this.timeout(0);
     let app;

tests/layout/test.misc.js

@@ -17,7 +17,6 @@
  * limitations under the License.
  */
 
-import lng from '../../src'
 import FlexTestUtils from "./src/FlexTestUtils.mjs";
 
 const flexTestUtils = new FlexTestUtils();

tests/shaders/test.grayscale.js

@@ -17,8 +17,6 @@
  * limitations under the License.
  */
 
-import lng from '../../src'
-
 describe('Shaders', function() {
     this.timeout(0);
 

tests/test.html

@@ -32,6 +32,7 @@
 
 <script>mocha.setup('bdd')</script>
 
+<script src="../dist/lightning.es5.js"></script>
 <script src="textures/test.textures.js" type="module"></script>
 <script src="textures/test.text.js" type="module"></script>
 <script src="keyhandler/test.keyhandler.js" type="module"></script>

tests/textures/test.text.js

@@ -17,8 +17,6 @@
  * limitations under the License.
  */
 
-import lng from '../../src'
-
 const EXAMPLE_TEXT =
 'Lorem ipsum dolor sit amet, consectetur adipiscing elit. Proin nibh augue, \
 suscipit a, scelerisque sed, lacinia in, mi. Cras vel lorem. Etiam pellentesque \

tests/textures/test.textures.js

@@ -17,8 +17,6 @@
  * limitations under the License.
  */
 
-import lng from '../../src'
-
 describe('textures', function() {
     this.timeout(0);