search

ABI-Team-30 / Fresnel-Forms

A Protégé plugin for generating and adapting MediaWiki user interfaces for source ontologies
http://is.cs.ou.nl/OWF/index.php5/Fresnel_Forms
3 stars 1 forks source link

jfresnel api voor Fresnel testen #17

Closed LloydRutledge closed 9 years ago

LloydRutledge commented 10 years ago

Deze test de Fresnel syntaxis. Je laad dus de opgeslagen Fresnel-bestand door deze API. Die zou den foutmeldingen geven. En die een Fresnel-alleen genormaliseerde versie van het triple-bestand maken. Deze test niet het systeemgedrag, zoals ABI28 deed met de Fresnel browser Longwell.

TeunTheunissen commented 9 years ago

De gerefactored versie naar svn gecommitted, revisie 219. Graag een review. Ik heb er een soort van controller-model-view structuur in aangebracht. De jFrame classe is de controller, het model bestaat uit het jenamodel en het fresneldocument, de view bestaat uit de tabs. Ik heb een aantal classe afgescheiden 'FresnelReaderMenuBar' en 'FresnelReaderTabbedPane' om de fanout en abstract data coupling te verkleinen. Dit is maar ten dele gelukt, de waarden zijn vermindert maar voldoen niet aan de checkstyle normen. Laat maar weten hoe je erover denkt, zo laten of verder refactoren. Groeten Teun

AlexMekkering commented 9 years ago

Code én de tool zelf zien er keurig uit :+1: en het doet absoluut wat het moet doen op dit moment. We kunnen dit in de toekomst wellicht gebruiken bij het verifiëren van met name de format styling en het toepassen van Groups. Expliciet geformatteerde properties worden niet goed herkend, maar dit lijkt een probleem van jfresnel zelf te zijn.

Als tip wil ik nog meegeven dat de target map niet gecommit hoeft te worden (kan opnieuw gebouwd worden vanuit de broncode) en eclipse-specifieke bestanden ook niet (bijv. .project en .classpath) omdat ze door eclipse zelf worden opgebouwd bij het importeren van een Maven project.