Generator: test adding entity "in between" existing entities.

Author: greenrobot-team

generator/integration-tests/README.md

@@ -24,4 +24,4 @@ used to further prepare the environment **before** code generation for the step
 ```
 Invalid argument(s): Failed to load dynamic library 'lib/objectbox.dll'
 ```
-Ensure `objectbox-c` is installed globally (required on Linux), or in the tested directory run [`../../../install.sh`](../../install.sh).
+Ensure `objectbox-c` is installed globally, or in the tested directory run [`../../../install.sh`](../../install.sh).

generator/integration-tests/changes/3-pre.dart

@@ -0,0 +1,45 @@
+import 'dart:io';
+
+void main() {
+  // Goals:
+  // * add a new entity lexicographically in between the existing ones to check
+  //   that IDs are not re-assigned.
+
+  File('lib/entities.dart').writeAsStringSync('''
+import 'package:objectbox/objectbox.dart';
+
+@Entity()
+class A {
+  int? id;
+
+  String? text1;
+
+  @Property(uid: 1003)
+  String? text2;
+
+  @Property(uid: 1004)
+  final relOne = ToOne<B>();
+
+  @Property(uid: 1005)
+  final relMany = ToMany<B>();
+
+  A();
+}
+
+@Entity()
+class A1 {
+  int? id;
+
+  A1();
+}
+
+@Entity(uid: 2000)
+class B {
+  int? id;
+
+  bool? value;
+
+  B();
+}
+  ''');
+}

generator/integration-tests/changes/3.dart

@@ -0,0 +1,66 @@
+import 'dart:io';
+
+import 'package:io/io.dart';
+import 'package:test/test.dart';
+
+import '../common.dart';
+import 'lib/lib.dart';
+import 'lib/objectbox.g.dart';
+
+void main() {
+  final defs = getObjectBoxModel();
+  final jsonModel = readModelJson('lib');
+  commonModelTests(defs, jsonModel);
+
+  test('ensure current model looks like expected', () {
+    final model = defs.model;
+
+    expect(model.entities.length, 3);
+
+    expect(model.entities[0].name, 'A');
+    expect(model.entities[0].properties.length, 4);
+    expect(model.entities[0].properties[0].name, 'id');
+    expect(model.entities[0].properties[1].name, 'text1');
+    expect(model.entities[0].properties[2].name, 'text2');
+    expect(model.entities[0].properties[3].name, 'relOneId');
+    expect(model.entities[0].properties[3].relationTarget, 'B');
+
+    expect(model.entities[0].relations.length, 1);
+    expect(model.entities[0].relations[0].name, 'relMany');
+    expect(model.entities[0].relations[0].targetId.toString(), '2:2000');
+
+    expect(model.entities[1].name, 'B');
+    expect(model.entities[1].properties.length, 2);
+    expect(model.entities[1].properties[0].name, 'id');
+    expect(model.entities[1].properties[1].name, 'value');
+
+    expect(model.entities[2].name, 'A1');
+    expect(model.entities[2].properties.length, 1);
+    expect(model.entities[2].properties[0].name, 'id');
+
+    expect(model.lastEntityId.toString(), model.entities[2].id.toString());
+
+    expect(jsonModel.retiredEntityUids.length, 0);
+    expect(jsonModel.retiredPropertyUids.length, 0);
+    expect(jsonModel.retiredIndexUids.length, 0);
+  });
+
+  /// test the data has been migrated from the previous version
+  test('data', () {
+    final srcDir = Directory('objectbox.2');
+    final tarDir = Directory('objectbox.3');
+
+    expect(srcDir.existsSync(), isTrue);
+    if (tarDir.existsSync()) tarDir.deleteSync(recursive: true);
+    copyPathSync(srcDir.path, tarDir.path);
+
+    final store = Store(defs, directory: tarDir.path);
+    final boxA = Box<A>(store);
+    final boxB = Box<B>(store);
+
+    expect(boxA.count(), 3);
+    expect(boxB.count(), 1);
+
+    store.close();
+  });
+}

generator/integration-tests/changes/4-pre.dart

@@ -28,6 +28,13 @@ class A {
   A();
 }
 
+@Entity()
+class A1 {
+  int? id;
+
+  A1();
+}
+
 @Entity(uid: 2000)
 class Renamed {
   int? id;

generator/integration-tests/changes/4.dart

@@ -1,9 +1,11 @@
 import 'dart:io';
+
 import 'package:io/io.dart';
 import 'package:test/test.dart';
+
+import '../common.dart';
 import 'lib/lib.dart';
 import 'lib/objectbox.g.dart';
-import '../common.dart';
 
 void main() {
   final defs = getObjectBoxModel();
@@ -13,7 +15,7 @@ void main() {
   test('ensure current model looks like expected', () {
     final model = defs.model;
 
-    expect(model.entities.length, 2);
+    expect(model.entities.length, 3);
 
     expect(model.entities[0].name, 'A');
     expect(model.entities[0].properties.length, 4);
@@ -32,7 +34,11 @@ void main() {
     expect(model.entities[1].properties[0].name, 'id');
     expect(model.entities[1].properties[1].name, 'value');
 
-    expect(model.lastEntityId.toString(), model.entities[1].id.toString());
+    expect(model.entities[2].name, 'A1');
+    expect(model.entities[2].properties.length, 1);
+    expect(model.entities[2].properties[0].name, 'id');
+
+    expect(model.lastEntityId.toString(), model.entities[2].id.toString());
 
     expect(jsonModel.retiredEntityUids.length, 0);
     expect(jsonModel.retiredPropertyUids.length, 0);
@@ -41,8 +47,8 @@ void main() {
 
   /// test the data has been migrated from the previous version and prepare new data for the next step
   test('data', () {
-    final srcDir = Directory('objectbox.2');
-    final tarDir = Directory('objectbox.3');
+    final srcDir = Directory('objectbox.3');
+    final tarDir = Directory('objectbox.4');
 
     expect(srcDir.existsSync(), isTrue);
     if (tarDir.existsSync()) tarDir.deleteSync(recursive: true);

generator/integration-tests/changes/5-pre.dart

@@ -16,5 +16,12 @@ class A {
 
   A();
 }
+
+@Entity()
+class A1 {
+  int? id;
+
+  A1();
+}
   ''');
 }

generator/integration-tests/changes/5.dart

@@ -1,6 +1,7 @@
 import 'package:test/test.dart';
-import 'lib/objectbox.g.dart';
+
 import '../common.dart';
+import 'lib/objectbox.g.dart';
 
 void main() {
   final defs = getObjectBoxModel();
@@ -10,7 +11,7 @@ void main() {
   test('ensure current model looks like expected', () {
     final model = defs.model;
 
-    expect(model.entities.length, 1);
+    expect(model.entities.length, 2);
 
     expect(model.entities[0].name, 'A');
     expect(model.entities[0].properties.length, 1);
@@ -19,7 +20,12 @@ void main() {
 
     expect(model.entities[0].relations.length, 0);
 
-    expect(model.lastEntityId.toString(), '2:2000');
+    expect(model.entities[1].name, 'A1');
+    expect(model.entities[1].properties.length, 1);
+    expect(model.entities[1].properties[0].name, 'id');
+
+    // Last ID should not change even after B was removed.
+    expect(model.lastEntityId.toString(), model.entities[1].id.toString());
     // lastRelationId is kept even if the relation itself is removed.
     expect(model.lastRelationId.toString(), '1:1005');