Fix Flutter package publish (#20)

Author: vdvukhzhilov

.github/workflows/publish.yml

@@ -52,6 +52,9 @@ jobs:
         working-directory: packages/allure_flutter_test
         run: flutter test test
 
+      - name: Publish dry-run
+        run: bash scripts/publish-dry-run.sh
+
   publish-commons:
     name: Publish allure_dart_commons
     if: startsWith(github.ref_name, 'allure_dart_commons-v')

.github/workflows/tests.yml

@@ -59,6 +59,9 @@ jobs:
         working-directory: packages/allure_flutter_test
         run: flutter analyze --no-fatal-infos
 
+      - name: Publish dry-run
+        run: bash scripts/publish-dry-run.sh
+
   build:
     name: Build (${{ matrix.name }})
     runs-on: ${{ matrix.os }}

CONTRIBUTING.md

@@ -56,6 +56,15 @@ dart analyze
 
 and, when Flutter is available, analyzes `allure_flutter_test`.
 
+Before a release, also validate pub.dev packaging:
+
+```bash
+bash scripts/publish-dry-run.sh
+```
+
+CI runs that dry-run on every pull request so unconstrained dependencies and
+similar publish blockers fail before tagging.
+
 ## Running Tests
 
 The default contributor path is the normal Dart and Flutter test runners.

packages/allure_flutter_test/.gitignore

@@ -1,2 +1,5 @@
 # LocalFileComparator failure diffs written next to goldens during test runs
 **/failures/
+
+# Accidental local packages created while experimenting with flutter create
+flutter_package_*/

packages/allure_flutter_test/.pubignore

@@ -7,3 +7,4 @@ pubspec_overrides.yaml
 build/
 allure-results/
 allure-report/
+flutter_package_*/

packages/allure_flutter_test/pubspec.yaml

@@ -18,10 +18,10 @@ dependencies:
     sdk: flutter
   integration_test:
     sdk: flutter
-  # Version is pinned by the Flutter SDK's own flutter_test dependency, so
-  # use `any` rather than a manually tracked constraint.
-  leak_tracker_flutter_testing: any
-  test_api: any
+  # Declared for pub.dev (unconstrained `any` is rejected). Keep these wide so
+  # Flutter SDK's own flutter_test pins can select the matching versions.
+  leak_tracker_flutter_testing: ^3.0.0
+  test_api: ^0.7.0
 
 dev_dependencies:
   lints: ^6.1.0

scripts/publish-dry-run.sh

@@ -0,0 +1,42 @@
+#!/usr/bin/env bash
+# Validate that packages can be published to pub.dev.
+# Fails on pub publish warnings (for example unconstrained dependencies).
+set -euo pipefail
+
+root="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+cd "${root}"
+
+overrides="${root}/packages/allure_flutter_test/pubspec_overrides.yaml"
+if [[ -f "${overrides}" ]]; then
+  echo "==> removing Flutter workspace overrides for publish dry-run"
+  rm -f "${overrides}"
+fi
+
+echo "==> dart pub get (workspace)"
+dart pub get
+
+for package in allure_dart_commons allure_dart_test; do
+  echo "==> dart pub publish --dry-run (${package})"
+  (
+    cd "packages/${package}"
+    dart pub publish --dry-run
+  )
+done
+
+echo "==> flutter pub get (allure_flutter_test)"
+(
+  cd packages/allure_flutter_test
+  flutter pub get
+)
+
+echo "==> flutter pub publish --dry-run (allure_flutter_test)"
+(
+  cd packages/allure_flutter_test
+  # Prefer flutter pub so local Flutter SDK resolution matches CI publishers.
+  if ! flutter pub publish --dry-run; then
+    echo "allure_flutter_test publish dry-run failed" >&2
+    exit 1
+  fi
+)
+
+echo "==> publish dry-run passed"