There should be a base class for functional tests in Feeds, as tests to be written for #2669974: Implement the CSV parser UI and [#] will likely have some overlap.
The base class should be called FeedsBrowserTestBase and be added to feeds/tests/src/Functional.

The class FeedsWebTestCase from the D7 version of Feeds can be used as reference.

Requirements

  • Feeds module should be enabled.
  • The setUp() method should add a content type called 'article'.
  • The setUp() method create an user with Feeds admin privileges.
  • There should be a method for adding a feed type.
  • There should be a method for adding mapping.
  • There should be a method for adding a feed.
  • There should be a method performing an import.

Optional

  • Port other methods from the FeedsWebTestCase class from the D7 version of Feeds.
CommentFileSizeAuthor
#3 feeds-base-test-class-2917925-3.patch1.07 KBmegachriz

Comments

MegaChriz created an issue. See original summary.

megachriz’s picture

megachriz
Primary language Dutch
commented
Parent issue: » #2917928: Plan for 8.x-3.0-alpha1 release
megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Active » Needs review
StatusFileSize
new feeds-base-test-class-2917925-3.patch1.07 KB

This is very very basic. It only declares that the Feeds module must be enabled. Note: also added a base class for Feeds kernel tests.

  • MegaChriz committed c4b32ce on 8.x-3.x 
    Issue #2917925 by MegaChriz: Added an 'empty' base class for Feeds...
megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Needs review » Active

Files have been added, but this issue is not yet resolved, so status back to "active".

megachriz’s picture

megachriz
Primary language Dutch
commented

With #2939503: Unpublish/Delete nodes not included in feed this issue is almost fixed.

There are just a few methods from the D7 version left to be ported. Among them, I think importUrl() and importFile() could be useful, but there are probably a few more methods worthwhile to port.

megachriz’s picture

megachriz
Primary language Dutch
commented
Title: Add a base class for Feeds functional tests » Port remaining methods from test base class in D7
megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Active » Fixed

It looks like this has been fixed a long time ago.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.