On #2920309-79: Add experimental module for Help Topics, @dawehner suggested moving this method to a unit test:

+++ b/core/modules/help_topics/tests/src/Functional/HelpTopicTest.php
@@ -0,0 +1,347 @@
+   */
+  public function testTopicBodyGetSet() {
CommentFileSizeAuthor
#5 2943921.patch11.16 KBjhodgdon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon created an issue. See original summary.

andypost’s picture

Issue tags: +Needs tests

Yes, coverage at unit level makes sense

jhodgdon’s picture

It really is a unit test. I think I just put it in the Functional test for convenience. It should be moved to its own test.

  • jhodgdon committed 077c6eb on 8.x-2.x
    Issue #2943921 by jhodgdon: Move body set/get and export/import tests...
jhodgdon’s picture

Status: Active » Fixed
FileSize
11.16 KB

It turned out we could move 2 methods from HelpTopicTest to a new HelpTopicKernelTest. So, I went ahead and committed this, since it is just moving test methods around (straight copy/paste), and the tests still pass locally. I'll also update the Core patch. Attaching patch here for the record...

Status: Fixed » Closed (fixed)

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