Add unit tests for the query and command builders
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 88/100
Direzione di ricerca
Leggi QuickbaseNet/Helpers/QuickbaseQueryBuilder.cs e QuickbaseNet/Helpers/QuickbaseCommandBuilder.cs, quindi usa QuickbaseNet.UnitTests/Tests/QuickbaseClientTests.cs come riferimento di stile. Aggiungi i due file di test dei builder indicati in QuickbaseNet.UnitTests/Tests/ e verifica ogni comportamento elencato creando richieste e asserendo gli oggetti restituiti. Esegui dotnet test --filter FullyQualifiedName~BuilderTests; il lavoro è completato quando tutti i casi passano e nessun file sotto QuickbaseNet/ è stato modificato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Neither builder has a test. The 7 existing tests cover the client constructor, QueryRecords and QuickbaseResult. Builder tests don't need the HTTP mock: build a request and assert on the object you get back.
What to do
Add QuickbaseQueryBuilderTests.cs and QuickbaseCommandBuilderTests.cs under QuickbaseNet.UnitTests/Tests/, in the style of the existing tests. Cover at least:
Build()copiesFrom,SelectandWhereinto the request.SortByandGroupBycalled twice keep both items, in order.Selectcalled twice keeps only the last list.- With nothing set,
SortBy,GroupByandOptionsare null. AddNewRecordtwice gives two entries inData, andReturnFieldsends up inFieldsToReturn.UpdateRecord(8, ...)adds field"3"with the string value"8"next to the configured fields.AddFieldssets every field, andAddFieldtwice on one ID keeps the last value.BuildDeleteCommandmapsForTabletoFromandWithDeletionCriteriatoWhere.
Test what the builders do today. If something looks wrong, say so in the PR instead of changing the builder.
Where to look
QuickbaseNet/Helpers/QuickbaseQueryBuilder.csQuickbaseNet/Helpers/QuickbaseCommandBuilder.cs(field 3 is set at line 65)QuickbaseNet.UnitTests/Tests/QuickbaseClientTests.csfor style
Done when
- Every case above has a passing test
- Nothing under
QuickbaseNet/changes
Testing it
dotnet test --filter FullyQualifiedName~BuilderTests
- Lingua principale
- C#
- Stelle
- 3
- Fork
- 2
- Merge medio
- 5h 53m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di hazeliscoding/quickbase-net
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`Forse già presa Una pull request collegata a questa issue è aperta o già unita. Apertaenhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
Update the package metadata that still points at ducksoopForse già presa @hazeliscoding l’ha presa 2 giorni fa. Aperta
hazeliscoding/quickbase-net#15 · 1 assegnatario ·
Tutte le issue di hazeliscoding/quickbase-net
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
KnifeLemon/Filee#363 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
area-Native untriaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
dotnet/machinelearning#7837 ·
I maintainer di solito rispondono entro 2 giorni