Skipping unknown shortcodes

[sebastienros]

Fixes OrchardCMS/OrchardCore#7715

[sebastienros]

@deanmarcussen opinion on the new behavior?

[deanmarcussen]

New behaviour sounds good.

No weirdness with nesting? i.e. some found, some not found?

[sebastienros]

New tests:
[hello] -> "Hello world!"
[upper] -> ToUpperCase
other tags not defined

        [InlineData("[hello][foo]", "Hello world![foo]")]
        [InlineData("[hello  ][foo]", "Hello world![foo]")]
        [InlineData("[foo][hello]", "[foo]Hello world!")]
        [InlineData("[upper][foo][/upper]", "[FOO]")]
        [InlineData("[upper] [ foo ] [/upper]", " [ FOO ] ")]
        [InlineData("[[upper] [[] foo []] [/upper]]", "[upper] [[] foo []] [/upper]")]
        [InlineData("[[[upper] [[] foo []] [/upper]]]", "[[upper] [[] foo []] [/upper]]")]
        [InlineData("[upper] [hello] [/upper]", " HELLO WORLD! ")]
        [InlineData("[foo arg1=blah] ", "[foo arg1=blah] ")]
        [InlineData("[  foo arg1=blah  ]", "[  foo arg1=blah  ]")]
        [InlineData("[  foo arg1=blah  ] [/foo]", "[  foo arg1=blah  ] [/foo]")]
        [InlineData("[/foo]", "[/foo]")]
        [InlineData(" [/ foo ] ", " [/ foo ] ")]

[sebastienros]

Your comment made me think and I added more tests, obviously found issues which are now fixed.