Skip to content

Merge pull request #69 from Adel-Atzouza/revert-68-tests #645

Merge pull request #69 from Adel-Atzouza/revert-68-tests

Merge pull request #69 from Adel-Atzouza/revert-68-tests #645

Triggered via push January 21, 2025 16:16
Status Success
Total duration 1m 38s
Artifacts

ci.yml

on: push

Annotations

43 warnings
build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
build: CargoHub/Models/APIKey.cs#L9
'APIKey.Id' hides inherited member 'BaseModel.Id'. Use the new keyword if hiding was intended.
build: CargoHub/Models/Warehouse.cs#L66
'Warehouse.CreatedAt' hides inherited member 'BaseModel.CreatedAt'. Use the new keyword if hiding was intended.
build: CargoHub/Controllers/WarehousesController.cs#L10
Parameter 'BaseStorageService<Warehouse> storage' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
build: CargoHub/Controllers/WarehousesController.cs#L10
Parameter 'ErrorHandler error' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
build: CargoHub/Models/Transfer.cs#L40
Non-nullable property 'ItemId' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build: CargoHub/Models/Shipment.cs#L63
Non-nullable property 'Reference' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build: CargoHub/Models/Shipment.cs#L64
Non-nullable property 'OrderStatus' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build: CargoHub/Models/Shipment.cs#L65
Non-nullable property 'Items' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build: CargoHub/Models/Shipment.cs#L41
Non-nullable property 'ShipmentType' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build: CargoHub/Models/Shipment.cs#L42
Non-nullable property 'ShipmentStatus' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
unit-tests
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
unit-tests: CargoHub/Models/APIKey.cs#L9
'APIKey.Id' hides inherited member 'BaseModel.Id'. Use the new keyword if hiding was intended.
unit-tests: CargoHub/Models/Warehouse.cs#L66
'Warehouse.CreatedAt' hides inherited member 'BaseModel.CreatedAt'. Use the new keyword if hiding was intended.
unit-tests: CargoHub/Controllers/WarehousesController.cs#L10
Parameter 'BaseStorageService<Warehouse> storage' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
unit-tests: CargoHub/Controllers/WarehousesController.cs#L10
Parameter 'ErrorHandler error' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
unit-tests: CargoHub/Controllers/TransfersController.cs#L10
Parameter 'ErrorHandler error' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
unit-tests: CargoHub/Controllers/TransfersController.cs#L29
Dereference of a possibly null reference.
unit-tests: CargoHub/Controllers/TransfersController.cs#L29
Dereference of a possibly null reference.
unit-tests: CargoHub/Services/ShipmentService.cs#L22
Argument of type 'IIncludableQueryable<Shipment, List<Order>?>' cannot be used for parameter 'source' of type 'IIncludableQueryable<Shipment, IEnumerable<Order>>' in 'IIncludableQueryable<Shipment, List<OrderItem>?> EntityFrameworkQueryableExtensions.ThenInclude<Shipment, Order, List<OrderItem>?>(IIncludableQueryable<Shipment, IEnumerable<Order>> source, Expression<Func<Order, List<OrderItem>?>> navigationPropertyPath)' due to differences in the nullability of reference types.
unit-tests: CargoHub/Services/ShipmentService.cs#L22
Argument of type 'IIncludableQueryable<Shipment, List<OrderItem>?>' cannot be used for parameter 'source' of type 'IIncludableQueryable<Shipment, IEnumerable<OrderItem>>' in 'IIncludableQueryable<Shipment, Item?> EntityFrameworkQueryableExtensions.ThenInclude<Shipment, OrderItem, Item?>(IIncludableQueryable<Shipment, IEnumerable<OrderItem>> source, Expression<Func<OrderItem, Item?>> navigationPropertyPath)' due to differences in the nullability of reference types.
unit-tests: CargoHub/Services/ShipmentService.cs#L32
Possible null reference assignment.
unit-tests: UnitTest/ItemLinesServiceTest.cs#L29
Possible null reference argument for parameter 'row' in 'Task<int?> BaseStorageService<ItemLine>.AddRow(ItemLine row)'.
unit-tests: UnitTest/ItemGroupServiceTest.cs#L29
Possible null reference argument for parameter 'row' in 'Task<int?> BaseStorageService<ItemGroup>.AddRow(ItemGroup row)'.
unit-tests: UnitTest/BaseItemStorageTests.cs#L36
Dereference of a possibly null reference.
unit-tests: UnitTest/BaseItemStorageTests.cs#L57
Converting null literal or possible null value to non-nullable type.
unit-tests: UnitTest/TransferServiceTests.cs#L90
Dereference of a possibly null reference.
unit-tests: UnitTest/Test_APIKey.cs#L74
Converting null literal or possible null value to non-nullable type.
unit-tests: UnitTest/WarehouseServiceTests.cs#L126
Dereference of a possibly null reference.
unit-tests: UnitTest/SupplierServiceTests.cs#L140
Dereference of a possibly null reference.
unit-tests: UnitTest/WarehouseServiceTests.cs#L14
Non-nullable field '_context' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
unit-tests: UnitTest/WarehouseServiceTests.cs#L15
Non-nullable field '_service' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
integration-tests
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
integration-tests: CargoHub/Models/APIKey.cs#L9
'APIKey.Id' hides inherited member 'BaseModel.Id'. Use the new keyword if hiding was intended.
integration-tests: CargoHub/Models/Warehouse.cs#L66
'Warehouse.CreatedAt' hides inherited member 'BaseModel.CreatedAt'. Use the new keyword if hiding was intended.
integration-tests: CargoHub/Controllers/WarehousesController.cs#L10
Parameter 'BaseStorageService<Warehouse> storage' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
integration-tests: CargoHub/Controllers/WarehousesController.cs#L10
Parameter 'ErrorHandler error' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
integration-tests: CargoHub/Models/Transfer.cs#L40
Non-nullable property 'ItemId' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
integration-tests: CargoHub/Models/Shipment.cs#L63
Non-nullable property 'Reference' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
integration-tests: CargoHub/Models/Shipment.cs#L64
Non-nullable property 'OrderStatus' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
integration-tests: CargoHub/Models/Shipment.cs#L65
Non-nullable property 'Items' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
integration-tests: CargoHub/Models/Shipment.cs#L41
Non-nullable property 'ShipmentType' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
integration-tests: CargoHub/Models/Shipment.cs#L42
Non-nullable property 'ShipmentStatus' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.