Azure / Azure/data-api-builder
Update SerializationDeserialization tests to validate field names and not field count.
Offen
testing
usability
validation
- Vorherrschende Sprache
- C#
- Sterne
- 1.5k
- Forks
- 370
- Ø Merge
- 3 T. 22 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
Examples in SerializationDeserialization tests validate field count with magic numbers. If this number were to change and fail tests, we have no way of knowing which fields the tests were checking for.
```csharp
Assert.AreEqual(fields, 8); // what are the 8 fields??
Assert.AreEqual(fields, 5); // what are the 5 fields?
```
https://github.com/Azure/data-api-builder/blob/e8d9fcb9ac1c2aa8b47808b0a658a09e092545e0/src/Service.Tests/Unittests/SerializationDeserializationTests.cs#L428-L445
Beitragsleitfaden
Bewertung
Dieses Issue wurde noch nicht bewertet.