Merge pull request #1067 from nextcloud/refactor/dev-container

CONTRIBUTING.md

@@ -38,8 +38,7 @@ You can read the full documentation at https://www.conventionalcommits.org.
 ## Tools
 We maintain a collection of scripts in `./tool/`.
 They range from setting up a local Nextcloud server (`./tool/dev.sh`) to generating assets.
-Be aware that due to limitations in Apache the dev server will crash when you resize the terminal window.
-    
+
 ## Monorepo
 For easier development we use a monorepo structure.
 This means that we have multiple packages in one git repository.

cspell.json

@@ -13,7 +13,7 @@
     "external",
     "packages/file_icons/lib/src/data.dart",
     "packages/neon_lints/lib",
-    "packages/nextcloud/test/files"
+    "tool/dev/static"
   ],
   "dictionaries": [
     "bash",

packages/nextcloud/test/core_test.dart

@@ -8,13 +8,10 @@ void main() {
   group(
     'core',
     () {
-      late DockerImage image;
-      setUpAll(() async => image = await getDockerImage());
-
       late DockerContainer container;
       late TestNextcloudClient client;
       setUp(() async {
-        container = await getDockerContainer(image);
+        container = await getDockerContainer();
         client = await getTestClient(container);
       });
       tearDown(() => container.destroy());

packages/nextcloud/test/dashboard_test.dart

@@ -7,13 +7,10 @@ void main() {
   group(
     'dashboard',
     () {
-      late DockerImage image;
-      setUpAll(() async => image = await getDockerImage());
-
       late DockerContainer container;
       late TestNextcloudClient client;
       setUp(() async {
-        container = await getDockerContainer(image);
+        container = await getDockerContainer();
         client = await getTestClient(container);
       });
       tearDown(() => container.destroy());

packages/nextcloud/test/helper.dart

@@ -156,8 +156,24 @@ Future<TestNextcloudClient> getTestClient(
   return client;
 }
 
-Future<DockerContainer> getDockerContainer(final DockerImage image, {final bool useApache = false}) async {
-  late ProcessResult result;
+Future<DockerContainer> getDockerContainer() async {
+  const dockerImageName = 'ghcr.io/nextcloud/neon/dev';
+
+  var result = await runExecutableArguments(
+    'docker',
+    [
+      'images',
+      '-q',
+      dockerImageName,
+    ],
+  );
+  if (result.exitCode != 0) {
+    throw Exception('Querying docker image failed: ${result.stderr}');
+  }
+  if (result.stdout.toString().isEmpty) {
+    throw Exception('Missing docker image $dockerImageName. Please build it using ./tool/build-dev-container.sh');
+  }
+
   late int port;
   while (true) {
     port = randomPort();
@@ -171,12 +187,7 @@ Future<DockerContainer> getDockerContainer(final DockerImage image, {final bool
         'host.docker.internal:host-gateway',
         '-p',
         '$port:80',
-        image,
-        if (!useApache) ...[
-          'php',
-          '-S',
-          '0.0.0.0:80',
-        ],
+        dockerImageName,
       ],
     );
     // 125 means the docker run command itself has failed which indicated the port is already used
@@ -195,35 +206,6 @@ Future<DockerContainer> getDockerContainer(final DockerImage image, {final bool
   );
 }
 
-typedef DockerImage = String;
-
-Future<DockerImage> getDockerImage() async {
-  const dockerImageName = 'nextcloud-neon-dev';
-
-  final inputStream = StreamController<List<int>>();
-  final process = runExecutableArguments(
-    'docker',
-    [
-      'build',
-      '-t',
-      dockerImageName,
-      '-f',
-      '-',
-      '../../tool',
-    ],
-    stdin: inputStream.stream,
-  );
-  inputStream.add(utf8.encode(File('../../tool/Dockerfile.dev').readAsStringSync()));
-  await inputStream.close();
-
-  final result = await process;
-  if (result.exitCode != 0) {
-    throw Exception('Failed to build docker image');
-  }
-
-  return dockerImageName;
-}
-
 class TestNextcloudUser {
   TestNextcloudUser(
     this.username,

packages/nextcloud/test/news_test.dart

@@ -3,7 +3,6 @@ import 'dart:async';
 import 'package:nextcloud/news.dart' as news;
 import 'package:nextcloud/nextcloud.dart';
 import 'package:test/test.dart';
-import 'package:universal_io/io.dart';
 
 import 'helper.dart';
 
@@ -11,30 +10,22 @@ void main() {
   group(
     'news',
     () {
-      late DockerImage image;
-      late HttpServer rssServer;
-      setUpAll(() async {
-        image = await getDockerImage();
-        rssServer = await getRssServer();
-      });
-      tearDownAll(() async => rssServer.close(force: true));
-
       late DockerContainer container;
       late TestNextcloudClient client;
       setUp(() async {
-        container = await getDockerContainer(image);
+        container = await getDockerContainer();
         client = await getTestClient(container);
       });
       tearDown(() => container.destroy());
 
       Future<DynamiteResponse<news.ListFeeds, void>> addWikipediaFeed([final int? folderID]) async =>
           client.news.addFeed(
-            url: 'http://host.docker.internal:${rssServer.port}/wikipedia.xml',
+            url: 'http://localhost/static/wikipedia.xml',
             folderId: folderID,
           );
 
       Future<DynamiteResponse<news.ListFeeds, void>> addNasaFeed() async => client.news.addFeed(
-            url: 'http://host.docker.internal:${rssServer.port}/nasa.xml',
+            url: 'http://localhost/static/nasa.xml',
           );
 
       test('Is supported', () async {
@@ -58,7 +49,7 @@ void main() {
         expect(response.body.starredCount, null);
         expect(response.body.newestItemId, isNotNull);
         expect(response.body.feeds, hasLength(1));
-        expect(response.body.feeds[0].url, 'http://host.docker.internal:${rssServer.port}/wikipedia.xml');
+        expect(response.body.feeds[0].url, 'http://localhost/static/wikipedia.xml');
 
         response = await client.news.listFeeds();
         expect(response.statusCode, 200);
@@ -67,7 +58,7 @@ void main() {
         expect(response.body.starredCount, 0);
         expect(response.body.newestItemId, isNotNull);
         expect(response.body.feeds, hasLength(1));
-        expect(response.body.feeds[0].url, 'http://host.docker.internal:${rssServer.port}/wikipedia.xml');
+        expect(response.body.feeds[0].url, 'http://localhost/static/wikipedia.xml');
       });
 
       test('Delete feed', () async {
@@ -401,7 +392,7 @@ void main() {
         expect(response.body.newestItemId, isNotNull);
         expect(response.body.feeds, hasLength(1));
         expect(response.body.feeds[0].folderId, 1);
-        expect(response.body.feeds[0].url, 'http://host.docker.internal:${rssServer.port}/wikipedia.xml');
+        expect(response.body.feeds[0].url, 'http://localhost/static/wikipedia.xml');
       });
 
       test('Mark folder as read', () async {
@@ -430,28 +421,3 @@ void main() {
     timeout: timeout,
   );
 }
-
-Future<HttpServer> getRssServer() async {
-  final wikipediaRss = File('test/files/wikipedia.xml').readAsStringSync();
-  final nasaRss = File('test/files/nasa.xml').readAsStringSync();
-  while (true) {
-    try {
-      final port = randomPort();
-      final server = await HttpServer.bind(InternetAddress.anyIPv6, port);
-      unawaited(
-        server.forEach((final request) async {
-          switch (request.uri.path) {
-            case '/wikipedia.xml':
-              request.response.write(wikipediaRss);
-            case '/nasa.xml':
-              request.response.write(nasaRss);
-            default:
-              request.response.statusCode = HttpStatus.badRequest;
-          }
-          await request.response.close();
-        }),
-      );
-      return server;
-    } catch (_) {}
-  }
-}

packages/nextcloud/test/notes_test.dart

@@ -9,13 +9,10 @@ void main() {
   group(
     'notes',
     () {
-      late DockerImage image;
-      setUpAll(() async => image = await getDockerImage());
-
       late DockerContainer container;
       late TestNextcloudClient client;
       setUp(() async {
-        container = await getDockerContainer(image);
+        container = await getDockerContainer();
         client = await getTestClient(container);
       });
       tearDown(() => container.destroy());

packages/nextcloud/test/notifications_test.dart

@@ -7,13 +7,10 @@ import 'helper.dart';
 
 void main() {
   group('notifications', () {
-    late DockerImage image;
-    setUpAll(() async => image = await getDockerImage());
-
     late DockerContainer container;
     late TestNextcloudClient client;
     setUp(() async {
-      container = await getDockerContainer(image);
+      container = await getDockerContainer();
       client = await getTestClient(
         container,
         username: 'admin',
@@ -103,13 +100,10 @@ void main() {
     });
 
     group('Push', () {
-      late DockerImage image;
-      setUpAll(() async => image = await getDockerImage());
-
       late DockerContainer container;
       late TestNextcloudClient client;
       setUp(() async {
-        container = await getDockerContainer(image);
+        container = await getDockerContainer();
         client = await getTestClient(
           container,
           username: 'admin',

packages/nextcloud/test/provisioning_api_test.dart

@@ -7,13 +7,10 @@ void main() {
   group(
     'provisioning_api',
     () {
-      late DockerImage image;
-      setUpAll(() async => image = await getDockerImage());
-
       late DockerContainer container;
       late TestNextcloudClient client;
       setUp(() async {
-        container = await getDockerContainer(image);
+        container = await getDockerContainer();
         client = await getTestClient(
           container,
           username: 'admin',

packages/nextcloud/test/settings_test.dart

@@ -5,29 +5,30 @@ import 'package:test/test.dart';
 
 import 'helper.dart';
 
-Future<void> main() async {
-  await run(await getDockerImage());
-}
-
-Future<void> run(final DockerImage image) async {
-  group('settings', () {
-    late DockerContainer container;
-    late TestNextcloudClient client;
-    setUp(() async {
-      container = await getDockerContainer(image);
-      client = await getTestClient(
-        container,
-        username: 'admin',
-      );
-    });
-    tearDown(() => container.destroy());
+void main() {
+  group(
+    'settings',
+    () {
+      late DockerContainer container;
+      late TestNextcloudClient client;
+      setUp(() async {
+        container = await getDockerContainer();
+        client = await getTestClient(
+          container,
+          username: 'admin',
+        );
+      });
+      tearDown(() => container.destroy());
 
-    group('Logs', () {
-      test('Download', () async {
-        final response = await client.settings.logSettings.download();
-        final logs = utf8.decode(response.body);
-        expect(logs, await container.nextcloudLogs());
+      group('Logs', () {
+        test('Download', () async {
+          final response = await client.settings.logSettings.download();
+          final logs = utf8.decode(response.body);
+          expect(logs, await container.nextcloudLogs());
+        });
       });
-    });
-  });
+    },
+    retry: retryCount,
+    timeout: timeout,
+  );
 }

packages/nextcloud/test/spreed_test.dart

@@ -12,16 +12,10 @@ void main() {
   group(
     'spreed',
     () {
-      late DockerImage image;
-      setUpAll(() async => image = await getDockerImage());
-
       late DockerContainer container;
       late TestNextcloudClient client1;
       setUp(() async {
-        container = await getDockerContainer(
-          image,
-          useApache: true,
-        );
+        container = await getDockerContainer();
         client1 = await getTestClient(container);
       });
       tearDown(() => container.destroy());

packages/nextcloud/test/uppush_test.dart

@@ -7,13 +7,10 @@ void main() {
   group(
     'uppush',
     () {
-      late DockerImage image;
-      setUpAll(() async => image = await getDockerImage());
-
       late DockerContainer container;
       late TestNextcloudClient client;
       setUp(() async {
-        container = await getDockerContainer(image);
+        container = await getDockerContainer();
         client = await getTestClient(
           container,
           username: 'admin',

packages/nextcloud/test/user_status_test.dart

@@ -7,13 +7,10 @@ void main() {
   group(
     'user_status',
     () {
-      late DockerImage image;
-      setUpAll(() async => image = await getDockerImage());
-
       late DockerContainer container;
       late TestNextcloudClient client;
       setUp(() async {
-        container = await getDockerContainer(image);
+        container = await getDockerContainer();
         client = await getTestClient(container);
       });
       tearDown(() => container.destroy());

packages/nextcloud/test/webdav_test.dart

@@ -26,14 +26,11 @@ void main() {
   group(
     'webdav',
     () {
-      late DockerImage image;
-      setUpAll(() async => image = await getDockerImage());
-
       late DockerContainer container;
       late TestNextcloudClient client;
 
       setUp(() async {
-        container = await getDockerContainer(image);
+        container = await getDockerContainer();
         client = await getTestClient(container);
       });
       tearDown(() => container.destroy());

tool/Dockerfile.dev

@@ -1,9 +1,7 @@
-ARG SERVER_VERSION=27.1.3@sha256:4ab3159b25310ffdb699f532d45f706d5d949100ed7ac11820d83467f29678b9
+ARG SERVER_VERSION=27.1.3-fpm-alpine@sha256:70ddd4abb804bc061a4f882a81cc62a219b262c0ed1784439e66b091ef26104b
 FROM nextcloud:$SERVER_VERSION
 
 WORKDIR /usr/src/nextcloud
-RUN chown -R www-data:www-data .
-USER www-data
 
 RUN ./occ maintenance:install --admin-pass admin --admin-email admin@example.com
 RUN ./occ config:system:set allow_local_remote_servers --value=true
@@ -30,8 +28,11 @@ RUN install_app_version spreed https://github.com/nextcloud-releases/spreed/rele
 RUN ./occ talk:turn:add turn,turns staticauth.openrelay.metered.ca:443 udp,tcp --secret openrelayprojectsecret
 
 RUN ./occ app:enable password_policy
-RUN (sh /entrypoint.sh php -S 0.0.0.0:8080 &) && \
-    until curl -s -o /dev/null http://localhost:8080/status.php; do true; done && \
+RUN (sh /entrypoint.sh php -S 0.0.0.0:80 &) && \
+    until curl -s -o /dev/null http://localhost/status.php; do true; done && \
     # Do not setup the demo user here
-    for user in admin user1 user2; do curl -u "$user:$user" -H "ocs-apirequest: true" -s -o /dev/null http://localhost:8080/ocs/v2.php/cloud/user; done
-COPY --chown=www-data:www-data overlay /usr/src/nextcloud/
+    for user in admin user1 user2; do curl -u "$user:$user" -H "ocs-apirequest: true" -s -o /dev/null http://localhost/ocs/v2.php/cloud/user; done
+COPY --chown=www-data:www-data dev/static /usr/src/nextcloud/static
+
+ENV PHP_CLI_SERVER_WORKERS=10
+CMD ["php", "-S", "0.0.0.0:80"]

tool/dev.sh

@@ -6,4 +6,13 @@ source tool/common.sh
 ./tool/build-dev-container.sh
 
 echo "Running development instance on http://localhost. To access it in an Android Emulator use http://10.0.2.2"
-docker run --rm -v nextcloud-neon-dev:/usr/src/nextcloud -v nextcloud-neon-dev:/var/www/html -p "80:80" --add-host=host.docker.internal:host-gateway "$(image_tag "dev:latest")"
+
+container="$(docker run -d --rm -v nextcloud-neon-dev:/usr/src/nextcloud -v nextcloud-neon-dev:/var/www/html -p "80:80" --add-host=host.docker.internal:host-gateway "$(image_tag "dev:latest")")"
+function cleanup() {
+    docker kill "$container"
+}
+trap cleanup EXIT
+
+docker logs -f "$container" &
+
+sleep infinity

tool/dev/install_app_version

@@ -1,5 +1,5 @@
-#!/bin/bash
-set -euxo pipefail
+#!/bin/sh
+set -euxo
 
 app="$1"
 url="$2"