0048563: Redesign org.openbravo.api.tests module - Openbravo Issue Tracking System

Openbravo Issue Tracking System - Retail Modules

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0048563

Retail Modules

Retail API

public

2022-02-09 17:58

2022-02-14 13:45

Reporter

caristu

Assigned To

Triage Platform Conn

Priority

high

Severity

minor

Reproducibility

have not tried

Status

acknowledged

Resolution

open

Platform

OS Version

Product Version

Target Version

Fixed in Version

Merge Request Status

Review Assigned To

OBNetwork customer

Support ticket

Regression level

Regression date

Regression introduced in release

Regression introduced by commit

Triggers an Emergency Pack

Summary

0048563: Redesign org.openbravo.api.tests module

Description

Currently the org.openbravo.api.tests module contains all the test for the common, retail and retail api for giftcards test that tests import and export mappings and also the different export filters. Note that most of these tests depends on having sample data installed to be able to retrieve the data properly.

This module definition is far from ideal because:

- We have to add a new dependency to every new module that needs to test their mappings.
- We can't distribute the tests in the CI jobs that ideally should be run. Currently all the tests are being executed in a CI job that ensures that all the dependencies of the test module are met.

Steps To Reproduce

Proposed Solution

Split this test module in separate modules:

- org.openbravo.api.tests: with just the testing infrastructure.
- org.openbravo.retail.api.tests: with the tests for the retail api mappings
- org.openbravo.retail.api.giftcards.tests: with the tests for the retail api for giftcards mappings
- the same for the rest of the modules that define mappings

Note that we are separating these tests from their modules mainly because they need sampledata to be executed. And we do not want to add a depedency in those modules with any sampledata module because it does not make sense.

Additional Information