diff --git a/src/Orchard.Tests/Environment/Extensions/ExtensionFoldersTests.cs b/src/Orchard.Tests/Environment/Extensions/ExtensionFoldersTests.cs index 2b36907a1..9fddee3d5 100644 --- a/src/Orchard.Tests/Environment/Extensions/ExtensionFoldersTests.cs +++ b/src/Orchard.Tests/Environment/Extensions/ExtensionFoldersTests.cs @@ -48,12 +48,15 @@ namespace Orchard.Tests.Environment.Extensions { } [Test] - public void NamesFromFoldersWithModuleTxtShouldBeListed() { + public void IdsFromFoldersWithModuleTxtShouldBeListed() { IExtensionFolders folders = new ModuleFolders(new[] { _tempFolderName }, new StubCacheManager(), new StubWebSiteFolder()); - var names = folders.AvailableExtensions().Select(d => d.Id); - Assert.That(names.Count(), Is.EqualTo(2)); - Assert.That(names, Has.Some.EqualTo("Sample1")); - Assert.That(names, Has.Some.EqualTo("Sample3")); + var ids = folders.AvailableExtensions().Select(d => d.Id); + Assert.That(ids.Count(), Is.EqualTo(5)); + Assert.That(ids, Has.Some.EqualTo("Sample1")); // Sample1 - obviously + Assert.That(ids, Has.Some.EqualTo("Sample3")); // Sample3 + Assert.That(ids, Has.Some.EqualTo("Sample4")); // Sample4 + Assert.That(ids, Has.Some.EqualTo("Sample6")); // Sample6 + Assert.That(ids, Has.Some.EqualTo("Sample7")); // Sample7 } [Test] @@ -61,7 +64,31 @@ namespace Orchard.Tests.Environment.Extensions { IExtensionFolders folders = new ModuleFolders(new[] { _tempFolderName }, new StubCacheManager(), new StubWebSiteFolder()); var sample1 = folders.AvailableExtensions().Single(d => d.Id == "Sample1"); Assert.That(sample1.Id, Is.Not.Empty); - Assert.That(sample1.Author, Is.EqualTo("Bertrand Le Roy")); + Assert.That(sample1.Author, Is.EqualTo("Bertrand Le Roy")); // Sample1 } - } + + [Test] + public void NamesFromFoldersWithModuleTxtShouldFallBackToIdIfNotGiven() { + IExtensionFolders folders = new ModuleFolders(new[] { _tempFolderName }, new StubCacheManager(), new StubWebSiteFolder()); + var names = folders.AvailableExtensions().Select(d => d.Name); + Assert.That(names.Count(), Is.EqualTo(5)); + Assert.That(names, Has.Some.EqualTo("Le plug-in français")); // Sample1 + Assert.That(names, Has.Some.EqualTo("This is another test.txt")); // Sample3 + Assert.That(names, Has.Some.EqualTo("Sample4")); // Sample4 + Assert.That(names, Has.Some.EqualTo("SampleSix")); // Sample6 + Assert.That(names, Has.Some.EqualTo("Sample7")); // Sample7 + } + + [Test] + public void PathsFromFoldersWithModuleTxtShouldFallBackAppropriatelyIfNotGiven() { + IExtensionFolders folders = new ModuleFolders(new[] { _tempFolderName }, new StubCacheManager(), new StubWebSiteFolder()); + var paths = folders.AvailableExtensions().Select(d => d.Path); + Assert.That(paths.Count(), Is.EqualTo(5)); + Assert.That(paths, Has.Some.EqualTo("Sample1")); // Sample1 - Id, Name invalid URL segment + Assert.That(paths, Has.Some.EqualTo("Sample3")); // Sample3 - Id, Name invalid URL segment + Assert.That(paths, Has.Some.EqualTo("ThisIs.Sample4")); // Sample4 - Path + Assert.That(paths, Has.Some.EqualTo("SampleSix")); // Sample6 - Name, no Path + Assert.That(paths, Has.Some.EqualTo("Sample7")); // Sample7 - Id, no Name or Path + } + } } \ No newline at end of file diff --git a/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample3/Module.txt b/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample3/Module.txt index c544d83a7..e1ab3bc71 100644 --- a/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample3/Module.txt +++ b/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample3/Module.txt @@ -1 +1,2 @@ Name: This is another test.txt +Description: But not really. \ No newline at end of file diff --git a/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample4/Module.txt b/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample4/Module.txt new file mode 100644 index 000000000..c22106a35 --- /dev/null +++ b/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample4/Module.txt @@ -0,0 +1,2 @@ +Path: ThisIs.Sample4 +Author: Bertrand Le Roy \ No newline at end of file diff --git a/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample5/Module.txt b/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample5/Module.txt new file mode 100644 index 000000000..43ecd0ca6 --- /dev/null +++ b/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample5/Module.txt @@ -0,0 +1,3 @@ +Name: AnotherSample +Path: Sample 5 +Author: Some One \ No newline at end of file diff --git a/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample6/Module.txt b/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample6/Module.txt new file mode 100644 index 000000000..787b1f0d6 --- /dev/null +++ b/src/Orchard.Tests/Environment/Extensions/FoldersData/Sample6/Module.txt @@ -0,0 +1,2 @@ +Name: SampleSix +Author: Some One \ No newline at end of file diff --git a/src/Orchard.Tests/Mvc/Routes/StandardExtensionRouteProviderTests.cs b/src/Orchard.Tests/Mvc/Routes/StandardExtensionRouteProviderTests.cs index 9954a080c..bdfc8eb4c 100644 --- a/src/Orchard.Tests/Mvc/Routes/StandardExtensionRouteProviderTests.cs +++ b/src/Orchard.Tests/Mvc/Routes/StandardExtensionRouteProviderTests.cs @@ -18,7 +18,9 @@ namespace Orchard.Tests.Mvc.Routes { Feature =new Feature { Descriptor=new FeatureDescriptor { Extension=new ExtensionDescriptor { - Name="Foo" + Id="Foo", + Name="A Foo Module", + Path="Foo" } } } @@ -28,7 +30,9 @@ namespace Orchard.Tests.Mvc.Routes { Feature =new Feature { Descriptor=new FeatureDescriptor { Extension=new ExtensionDescriptor { - Name="Bar" + Id="Bar", + Name="Bar", + Path="BarBar" } } } @@ -46,9 +50,9 @@ namespace Orchard.Tests.Mvc.Routes { var fooRoute = routes.Select(x => x.Route).OfType() .Single(x => x.Url == "Foo/{controller}/{action}/{id}"); var barAdmin = routes.Select(x => x.Route).OfType() - .Single(x => x.Url == "Admin/Bar/{action}/{id}"); + .Single(x => x.Url == "Admin/BarBar/{action}/{id}"); var barRoute = routes.Select(x => x.Route).OfType() - .Single(x => x.Url == "Bar/{controller}/{action}/{id}"); + .Single(x => x.Url == "BarBar/{controller}/{action}/{id}"); Assert.That(fooAdmin.DataTokens["area"], Is.EqualTo("Long.Name.Foo")); Assert.That(fooRoute.DataTokens["area"], Is.EqualTo("Long.Name.Foo")); diff --git a/src/Orchard.Tests/Orchard.Framework.Tests.csproj b/src/Orchard.Tests/Orchard.Framework.Tests.csproj index 33413970a..bee36fa25 100644 --- a/src/Orchard.Tests/Orchard.Framework.Tests.csproj +++ b/src/Orchard.Tests/Orchard.Framework.Tests.csproj @@ -335,7 +335,18 @@ true - + + + + + + + + + + + +