Skip to main content

Notice: this Wiki will be going read only early in 2024 and edits will no longer be possible. Please see: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/wikis/Wiki-shutdown-plan for the plan.

Jump to: navigation, search

CDO Development Guidelines

Revision as of 04:10, 9 August 2010 by Martin.fluegge.gmx.de (Talk | contribs) (General Requirements)

Coding Guidlines

This page is not yet finished. Do not take these guidelines for granted until the note is removed!

Basic Coding Guidelines

The following Guide describes the basic requirements for creating CDO source code. Not all of these guidlines can be coverred by the auto-format for Java source Code (Strg+Shift+F). If you like to write an extension that allows comfortably adjusting the code regarding these Guidelines you are welcome.

Members

  • Make sure your members are always organized with descending accessors (public, protected, private).
  • Group them logically. Members that fit together should be written under each other.

Methods

  • Make sure your methodes are always organized with descending accessors (public, protected, private)

Getter and Setter

Guideline for writing test cases

Ecore models

  • if you write ecore models, make sure that your model's name matches the last segment of the plug-in name (e.g. org.eclipse.emf.cdo.dawn.examples.acore -->acore.ecore)
  • make sure that the gemodels default base packages matches the plugin name (without the last segment)

Legacy Mode

In the CDO test suite the Legacy Mode is always activated. This means that you do not need set this mode for every test case. But this also means that you need to deactivate it if you want to execute tests without legacy.

Please always use CDOUtil.getCDOObject() and CDOUtil.getEObject() to get either the CDOObject or the expected internal instance, no matter whether you are using legacy or not. This will make the test compatible to all legacy test cases and avoids ugly ClassCastExceptions.

When using dynamic EObjects please do not use CDOUtil.prepareDynamicEPackage(dynamicMapEPackage) if you want to make this test legacy aware. Otherwise the package will be native even if the Legacy test suite is executed. Please use somtehing like the following statement to make sure that you have prepared dynamic legacy objects.

if(!CDOUtil.isLegacyModeDefault())
{
CDOUtil.prepareDynamicEPackage(dynamicMapEPackage);
}

Back to the top