Corrected the use of special string container classes in cases when a

  single tag may contain strings with different containers; such as
  the <template> tag, which may contain both TemplateString objects
  and Comment objects. [bug=1913406]

1 files changed, 13 insertions, 2 deletions

diff --git a/bs4/tests/test_tree.py b/bs4/tests/test_tree.py
index 2246346..1bd1577 100644
--- a/bs4/tests/test_tree.py
+++ b/bs4/tests/test_tree.py
@@ -1911,14 +1911,25 @@ class TestNavigableStringSubclasses(SoupTest):
         soup = self.soup(
             "<template>Some text<p>In a tag</p></template>Some text outside"
         )
-        assert all(isinstance(x, TemplateString) for x in soup.template.strings)
-
+        assert all(
+            isinstance(x, TemplateString)
+            for x in soup.template._all_strings(types=None)
+        )
+        
         # Once the <template> tag closed, we went back to using
         # NavigableString.
         outside = soup.template.next_sibling
         assert isinstance(outside, NavigableString)
         assert not isinstance(outside, TemplateString)
 
+        # The TemplateString is also unusual because it can contain
+        # NavigableString subclasses of _other_ types, such as
+        # Comment.
+        markup = b"<template>Some text<p>In a tag</p><!--with a comment--></template>"
+        soup = self.soup(markup)
+        self.assertEqual(markup, soup.template.encode("utf8"))
+
+        
 class TestSoupSelector(TreeTest):
 
     HTML = """