Merge remote-tracking branch 'origin/development' into feature/88-neue-datenbank-struktur

.gitea/workflows/push.yaml

@@ -85,7 +85,6 @@ jobs:
           strategy: 'patch'
           path: './pubspec.yaml'
 
-
       - name: Commit version update
         env:
           GITEA_TOKEN: ${{ secrets.BOT_TOKEN }}
@@ -107,6 +106,7 @@ jobs:
         with:
           fetch-depth: 0
           token: ${{ secrets.BOT_TOKEN }}
+          ref: ${{ gitea.ref_name }}
 
       # Required for Flutter action
       - name: Install jq

lib/presentation/views/main_menu/settings_view/licenses/oss_licenses.dart

@@ -1396,13 +1396,13 @@ THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.''',
   );
 
-/// cross_file 0.3.5+1
+/// cross_file 0.3.5+2
 const _cross_file = Package(
     name: 'cross_file',
     description: 'An abstraction to allow working with files across multiple platforms.',
     repository: 'https://github.com/flutter/packages/tree/main/packages/cross_file',
     authors: [],
-    version: '0.3.5+1',
+    version: '0.3.5+2',
     spdxIdentifiers: ['BSD-3-Clause'],
     isMarkdown: false,
     isSdk: false,
@@ -1628,13 +1628,13 @@ THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.''',
   );
 
-/// dbus 0.7.11
+/// dbus 0.7.12
 const _dbus = Package(
     name: 'dbus',
     description: 'A native Dart implementation of the D-Bus message bus client. This package allows Dart applications to directly access services on the Linux desktop.',
     homepage: 'https://github.com/canonical/dbus.dart',
     authors: [],
-    version: '0.7.11',
+    version: '0.7.12',
     spdxIdentifiers: ['MPL-2.0'],
     isMarkdown: false,
     isSdk: false,
@@ -2015,7 +2015,7 @@ Exhibit B - "Incompatible With Secondary Licenses" Notice
   defined by the Mozilla Public License, v. 2.0.''',
   );
 
-/// dio 5.9.0
+/// dio 5.9.1
 const _dio = Package(
     name: 'dio',
     description: '''A powerful HTTP networking package,
@@ -2026,7 +2026,7 @@ Custom adapters, Transformers, etc.
     homepage: 'https://github.com/cfug/dio',
     repository: 'https://github.com/cfug/dio/blob/main/dio',
     authors: [],
-    version: '5.9.0',
+    version: '5.9.1',
     spdxIdentifiers: ['MIT'],
     isMarkdown: false,
     isSdk: false,
@@ -2497,14 +2497,14 @@ THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.''',
   );
 
-/// file_picker 10.3.8
+/// file_picker 10.3.10
 const _file_picker = Package(
     name: 'file_picker',
     description: 'A package that allows you to use a native file explorer to pick single or multiple absolute file paths, with extension filtering support.',
     homepage: 'https://github.com/miguelpruivo/plugins_flutter_file_picker',
     repository: 'https://github.com/miguelpruivo/flutter_file_picker',
     authors: [],
-    version: '10.3.8',
+    version: '10.3.10',
     spdxIdentifiers: ['MIT'],
     isMarkdown: false,
     isSdk: false,
@@ -2947,13 +2947,13 @@ THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.''',
   );
 
-/// hooks 1.0.0
+/// hooks 1.0.1
 const _hooks = Package(
     name: 'hooks',
     description: 'A library that contains a Dart API for the JSON-based protocol for `hook/build.dart` and `hook/link.dart`.',
     repository: 'https://github.com/dart-lang/native/tree/main/pkgs/hooks',
     authors: [],
-    version: '1.0.0',
+    version: '1.0.1',
     spdxIdentifiers: ['BSD-3-Clause'],
     isMarkdown: false,
     isSdk: false,
@@ -3271,13 +3271,13 @@ THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.''',
   );
 
-/// json_annotation 4.9.0
+/// json_annotation 4.10.0
 const _json_annotation = Package(
     name: 'json_annotation',
     description: 'Classes and helper functions that support JSON code generation via the `json_serializable` package.',
     repository: 'https://github.com/google/json_serializable.dart/tree/master/json_annotation',
     authors: [],
-    version: '4.9.0',
+    version: '4.10.0',
     spdxIdentifiers: ['BSD-3-Clause'],
     isMarkdown: false,
     isSdk: false,
@@ -4085,13 +4085,13 @@ THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.''',
   );
 
-/// objective_c 9.2.4
+/// objective_c 9.3.0
 const _objective_c = Package(
     name: 'objective_c',
     description: 'A library to access Objective C from Flutter that acts as a support library for package:ffigen.',
     repository: 'https://github.com/dart-lang/native/tree/main/pkgs/objective_c',
     authors: [],
-    version: '9.2.4',
+    version: '9.3.0',
     spdxIdentifiers: ['BSD-3-Clause'],
     isMarkdown: false,
     isSdk: false,
@@ -5869,13 +5869,13 @@ THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.''',
   );
 
-/// source_span 1.10.1
+/// source_span 1.10.2
 const _source_span = Package(
     name: 'source_span',
     description: 'Provides a standard representation for source code locations and spans.',
     repository: 'https://github.com/dart-lang/tools/tree/main/pkgs/source_span',
     authors: [],
-    version: '1.10.1',
+    version: '1.10.2',
     spdxIdentifiers: ['BSD-3-Clause'],
     isMarkdown: false,
     isSdk: false,
@@ -7499,12 +7499,12 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.''',
   );
 
-/// tallee 0.0.13+247
+/// tallee 0.0.15+249
 const _tallee = Package(
     name: 'tallee',
     description: 'Tracking App for Card Games',
     authors: [],
-    version: '0.0.13+247',
+    version: '0.0.15+249',
     spdxIdentifiers: ['LGPL-3.0'],
     isMarkdown: false,
     isSdk: false,

pubspec.yaml

@@ -1,7 +1,7 @@
 name: tallee
 description: "Tracking App for Card Games"
 publish_to: 'none'
-version: 0.0.14+248
+version: 0.0.15+249
 
 environment:
   sdk: ^3.8.1

test/db_tests/aggregates/match_test.dart

@@ -122,20 +122,16 @@ void main() {
       } else {
         fail('Group is null');
       }
-      if (result.players != null) {
+      expect(result.players.length, testMatch1.players.length);
-        expect(result.players!.length, testMatch1.players!.length);
 
-        for (int i = 0; i < testMatch1.players!.length; i++) {
+      for (int i = 0; i < testMatch1.players.length; i++) {
-          expect(result.players![i].id, testMatch1.players![i].id);
+        expect(result.players[i].id, testMatch1.players[i].id);
-          expect(result.players![i].name, testMatch1.players![i].name);
+        expect(result.players[i].name, testMatch1.players[i].name);
         expect(
-            result.players![i].createdAt,
+          result.players[i].createdAt,
-            testMatch1.players![i].createdAt,
+          testMatch1.players[i].createdAt,
         );
       }
-      } else {
-        fail('Players is null');
-      }
     });
 
     // Verifies that multiple matches can be added and retrieved with correct groups and players.
@@ -191,19 +187,15 @@ void main() {
         }
 
         // Players-Checks
-        if (testMatch.players != null) {
+        expect(match.players.length, testMatch.players.length);
-          expect(match.players!.length, testMatch.players!.length);
+        for (int i = 0; i < testMatch.players.length; i++) {
-          for (int i = 0; i < testMatch.players!.length; i++) {
+          expect(match.players[i].id, testMatch.players[i].id);
-            expect(match.players![i].id, testMatch.players![i].id);
+          expect(match.players[i].name, testMatch.players[i].name);
-            expect(match.players![i].name, testMatch.players![i].name);
           expect(
-              match.players![i].createdAt,
+            match.players[i].createdAt,
-              testMatch.players![i].createdAt,
+            testMatch.players[i].createdAt,
           );
         }
-        } else {
-          expect(match.players, null);
-        }
       }
     });
 

test/db_tests/entities/game_test.dart

@@ -132,7 +132,7 @@ void main() {
       expect(allGames.length, 1);
     });
 
-    // Verifies that a game with null optional fields can be added and retrieved.
+    // Verifies that a game with empty optional fields can be added and retrieved.
     test('addGame handles game with null optional fields', () async {
       final gameWithNulls = Game(name: 'Simple Game', ruleset: Ruleset.lowestScore, description: 'A simple game', color: GameColor.green, icon: '');
       final result = await database.gameDao.addGame(game: gameWithNulls);
@@ -144,7 +144,7 @@ void main() {
       expect(fetchedGame.name, 'Simple Game');
       expect(fetchedGame.description, 'A simple game');
       expect(fetchedGame.color, GameColor.green);
-      expect(fetchedGame.icon, isNull);
+      expect(fetchedGame.icon, '');
     });
 
     // Verifies that multiple games can be added at once using addGamesAsList.

test/db_tests/relationships/player_match_test.dart

@@ -140,7 +140,7 @@ void main() {
     test('Removing player from match works correctly', () async {
       await database.matchDao.addMatch(match: testMatchOnlyPlayers);
 
-      final playerToRemove = testMatchOnlyPlayers.players![0];
+      final playerToRemove = testMatchOnlyPlayers.players[0];
 
       final removed = await database.playerMatchDao.removePlayerFromMatch(
         playerId: playerToRemove.id,
@@ -151,9 +151,9 @@ void main() {
       final result = await database.matchDao.getMatchById(
         matchId: testMatchOnlyPlayers.id,
       );
-      expect(result.players!.length, testMatchOnlyPlayers.players!.length - 1);
+      expect(result.players.length, testMatchOnlyPlayers.players.length - 1);
 
-      final playerExists = result.players!.any(
+      final playerExists = result.players.any(
             (p) => p.id == playerToRemove.id,
       );
       expect(playerExists, false);
@@ -164,18 +164,14 @@ void main() {
       await database.matchDao.addMatch(match: testMatchOnlyPlayers);
       final players = await database.playerMatchDao.getPlayersOfMatch(
         matchId: testMatchOnlyPlayers.id,
-      );
+      ) ?? [];
-
-      if (players == null) {
-        fail('Players should not be null');
-      }
 
       for (int i = 0; i < players.length; i++) {
-        expect(players[i].id, testMatchOnlyPlayers.players![i].id);
+        expect(players[i].id, testMatchOnlyPlayers.players[i].id);
-        expect(players[i].name, testMatchOnlyPlayers.players![i].name);
+        expect(players[i].name, testMatchOnlyPlayers.players[i].name);
         expect(
           players[i].createdAt,
-          testMatchOnlyPlayers.players![i].createdAt,
+          testMatchOnlyPlayers.players[i].createdAt,
         );
       }
     });