fix: isEveryDefined() now throws NotInitializedError before load()

Author: java-james

CHANGELOG.md

@@ -10,6 +10,7 @@ Release notes are available on [github][notes].
 
 # 6.0.1
 
+- [fix] `isEveryDefined()` now throws `NotInitializedError` before `load()`, consistent with all other read APIs
 - [fix] Replace `assert()` with explicit `if`/`throw` in `getInt()`, `getDouble()`, and `getBool()` so null-safety checks are enforced in release builds
 - Error messages now include the variable name for easier debugging
 - [fix] `load(isOptional: true)` no longer discards successfully loaded base file when an override file is missing or empty (fixes #70, #93, #101, #125)

README.md

@@ -204,8 +204,9 @@ export EXPORTED=hello
 **Parsing rules:**
 - Lines without `=` are ignored.
 - `#` starts a comment when it appears outside quoted values.
-- Double-quoted values expand `\n` to newlines and interpolate `$VAR` / `${VAR}`.
+- Double-quoted values unescape `\"` and expand `\n` to newlines, and interpolate `$VAR` / `${VAR}`.
 - Single-quoted values do not interpolate variables. Escaped single quotes (`\'`) are unescaped.
+- Use `\$` for a literal dollar sign in unquoted or double-quoted values.
 - Undefined variables interpolate to an empty string.
 - Unquoted values are trimmed of surrounding whitespace.
 - If the same key appears multiple times anywhere in the processed input, the first occurrence wins (see [value precedence](#value-precedence)).

lib/src/dotenv.dart

@@ -205,11 +205,10 @@ class DotEnv {
     _isInitialized = true;
   }
 
-  /// True if all supplied variables have nonempty value; false otherwise.
-  /// Differs from [containsKey](dart:core) by excluding null values.
-  /// Note [load] should be called first.
+  /// Returns true if all supplied variables have non-empty values.
+  /// Throws [NotInitializedError] if called before [load] or [loadFromString].
   bool isEveryDefined(Iterable<String> vars) =>
-      vars.every((k) => _envMap[k]?.isNotEmpty ?? false);
+      vars.every((k) => env[k]?.isNotEmpty ?? false);
 
   Future<List<String>> _getEntriesFromFile(String filename) async {
     try {

test/dotenv_test.dart

@@ -177,6 +177,31 @@ void main() {
     });
   });
 
+  group('isEveryDefined', () {
+    test('returns true when all variables exist with non-empty values', () {
+      dotenv.loadFromString(envString: 'A=1\nB=2\nC=3');
+      expect(dotenv.isEveryDefined(['A', 'B', 'C']), isTrue);
+    });
+
+    test('returns false when a variable is missing', () {
+      dotenv.loadFromString(envString: 'A=1\nB=2');
+      expect(dotenv.isEveryDefined(['A', 'B', 'MISSING']), isFalse);
+    });
+
+    test('returns false when a variable has an empty value', () {
+      dotenv.loadFromString(envString: 'A=1\nB=');
+      expect(dotenv.isEveryDefined(['A', 'B']), isFalse);
+    });
+
+    test('throws NotInitializedError before initialization', () {
+      final fresh = DotEnv();
+      expect(
+        () => fresh.isEveryDefined(['A']),
+        throwsA(isA<NotInitializedError>()),
+      );
+    });
+  });
+
   group('isOptional override loading', () {
     test(
         'loadFromString with valid base and empty override preserves base vars',