Skip to content

Tests dependencies #159

Description

@Youssef1313

I haven't yet dug into reading the source code carefully, but as a quick note, I see the Tests project depends on the Editor project.

I believe the Editor project should be only responsible for the "Web" things. Hence, it's just a high-level layer that depends on the lower-level components.

Given that, I don't think the Tests project should have a dependency on this web project.

The needed dependency could be for one of the following two reasons:

  1. The tests actually test things related to the web project. In that case, I think there should be a new test project for integration tests.
  2. The web project itself contains code that should have been in a different layer, causing the Tests project to have a dependency on it just to test things that aren't actually related to web.

Please, correct me if I'm wrong.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions