Crucible - Comment Search

Accordion

Defect Metrics

Statistics on the results

Ranking

Defect Report

Classification

Defect Report

1077 Defects:
435 major defects, and
642 other defects
from 4201 comments

Search Criteria

Search for comments

Review	Name	Created	Custom Fields	Content
CR-18		10 Jan 2017		Niet geverifieerde mocks mogen een andere benaming hebben.. stub of dummy.
CR-18		10 Jan 2017		testnaam mag meer informatie bevatten.
CR-18		10 Jan 2017		Testnaam mag meer informatie bevatten zodat het doel duidelijker is.
CR-18		10 Jan 2017		Testnaam / Methodnaam mag meer informatie bevatten.
CR-18		10 Jan 2017		XMLCorrellaterMock wordt niet geverifieerd. Deze mag Dummy worden of Stub.
CR-18		10 Jan 2017		Casing... #dim en ...RepoMock.
CR-18		10 Jan 2017		Duplicate code testdata mag gemeenschappelijk worden voor elke test.
CR-18		10 Jan 2017		Graag uitleg over deze test. Vanwaar één uur ? Kunnen er moeilijkheden zijn bij het verwerken in dit geval ?
CR-18		10 Jan 2017		writes nog nodig ?
CR-18		10 Jan 2017		Naam mag voluit : ...Repository.cls
CR-18		10 Jan 2017		Eens navragen waar persistent klassen komen te staan. bv onder /Data/
CR-18		10 Jan 2017		Method met veel lijnen opdelen in kleinere methods.
CR-18		10 Jan 2017		Bestaat er een BlumProductRepository die %ExistsId en %OpenId implementeert ?
CR-18		10 Jan 2017		Kleinere methods met minder lijnen.
CR-16		10 Jan 2017		Inderdaad. In Admin zal op de KAL lijst de verwachte leverweek de huidige week zijn.
CR-18		10 Jan 2017		Een Vrachtwagen kan uit meerdere despatch berichten bestaan. na een bepaalde duur zijn zijn we zeker dat de bestanden van een transport allemaal binnen zijn, en kunnen we de vrachtwagen verwerken
CR-18		10 Jan 2017		Niet dat ik weet.
CR-18		10 Jan 2017		Was zo besproken met [~JoC]
CR-7		12 Jan 2017		Uitzoeken landcodes ik maak hiervoor een kaartje aan. -> besproken met PVR. Idee om landen aan te bieden net zoals bij unishop is ok. Wordt later nog uitgewerkt.
CR-23		12 Jan 2017		set Kenmerken = ##class(Tools.Mock.ArgumentMatchers.DataTransferObjectMatcher).%New([StandaardKenmerken])
CR-23		12 Jan 2017		EdiConverter en LegacyConverter etc... samen nemen in klasse WSimpl.Vhisie4.Winkelkar.EconConfiguratieConverter.Convert(EconStream) As APPS.EC.Winkelkar.dto.MaatwerkLijn
CR-23		12 Jan 2017		Niet nodig om de bestaat via ExterneId op te roepen de GeefViaID is voldoende .. deze moeten dan wel de foutmelding encapsularen en zeggen dat er geen winkelkar bestaat met id.
CR-23		12 Jan 2017		GeefWinkelkarIdViaExterneId mag eigenlijk weg en het zou gewoon een GeefViaID mogen zijn waarvan je een offerte zal terug krijgen ( omdat voorlopig de repository geen notie hebben van wat ze moeten teruggeven , ( Strings, DataObjecten of Objecten ) op termijn zal het wel lukken als we de snelheid moeten aanpakken , maar voorlopig zou ik daar geen rekening met houden
CR-23		12 Jan 2017		#dim Lijn niet nodig
CR-23		12 Jan 2017		method niet nodig -> standaardlade kenmerken gebruiken
CR-23		12 Jan 2017		Request mag in-line gemaakt worden in de test zodat de test op zich duidelijk is
CR-23		12 Jan 2017		#dim lijn niet nodig
CR-23		12 Jan 2017		niet nodig
CR-23		12 Jan 2017		niet nodig
CR-23		12 Jan 2017		Afhankelijk van het antwoord van intracto maar ik vermoed dat deze er terug af mogen
CR-23		13 Jan 2017		Property EconConfiguratie As TECH.String (XMLNAME = "IngegevenKenmerken") [ Required ];
CR-21		13 Jan 2017		GeefViaID method gebruiken daar de WSimpl.Vhisie4.Winkelkar.WinkelkarRepos.. geen notie hebben van externeid in hun interface enkel in de implementatie
CR-21		13 Jan 2017		BestaatViaID WSimpl.Vhisie4.Winkelkar.WinkelkarRepos.. geen notie hebben van externeid in hun interface enkel in de implementatie
CR-21		13 Jan 2017		Controles zijn niet nodig daar de GeefViaId op de winkelkar repo en de lijn repo de juiste foutmeldingen zullen geven
CR-21		13 Jan 2017		BestaatViaID
CR-21		13 Jan 2017		GeefViaID
CR-21		13 Jan 2017		Lijkt mij overbodig
CR-21		13 Jan 2017		Lijkt mij overbodig
CR-21		13 Jan 2017		Lijkt mij overbodig
CR-21		13 Jan 2017	Ranking: Minor Classification: Risk-prone	Dit soort type testen schrijf je ofwel met echte data - data opzetten en kijken of de repo zijn ding doet . ofwel met een spy die op zijn minst de Query prepared - zodat we zeker zijn dat het query statement kan uitgevoerd worden. Het gevaar van dit soort testen is dat je uw Query statement in uw code gaat overkopieëren naar uw testen en als er een fout in het statement zit tabel naam dat bijvoorbeeld niet meer bestaat dat merk je het niet op.
CR-21		13 Jan 2017		Ik zou de naam van variabele renamen naar Service ofwel moeten we de methods renamen naar Invoke.. Want het is nu een beetje onduidelijk VerwijderLijn.VerwijderLijn .. klikt een beetje raar ofwel VerwijderLijnService.VerwijderLijn( ofwel VerwijderLijn.Invoke(
CR-21		13 Jan 2017		Hier zou ik gaan voor de WinkelkarLijnRepositoryMock ipv de LijnRepositoryMock .. als we dan toch voluit gaan...
CR-21		13 Jan 2017		Foutmelding is beter Winkelkar met id 'WinkelkarId' bestaat niet.
CR-21		13 Jan 2017	Classification: Extra (superfluous) Ranking: Minor	Overbodig dus
CR-30		13 Jan 2017	Classification: Risk-prone Ranking: Major	Geen defaults in conversies...
CR-30		13 Jan 2017	Ranking: Minor Classification: Extra (superfluous)	overbodig dus
CR-29		13 Jan 2017	Classification: Not conforming to standards Ranking: Major	implementatie klasse worden niet rechtstreeks genewed halfweg de code
CR-29		13 Jan 2017	Ranking: Minor Classification: Not conforming to standards	Waarom %New() van de OfferteService .. lijkt mij niet mockbaar dus ook niet deftig testbaar .
CR-29		13 Jan 2017	Ranking: Minor Classification: Not conforming to standards	GetWsOrdFromWWW - Out en natuurlijk ook de In
CR-33		13 Jan 2017	Ranking: Minor Classification: Not conforming to standards	enu's

Atlassian FishEye analysis with Crucible code review. (Version:4.2.1 Build:20161109135523 2016-11-09) - Administration - Page generated 2025-09-02 05:02 +0200