Permalink
Browse files

Remove public methods for adding items to groups. Groups don't work p…

…roperly.
  • Loading branch information...
1 parent c34f93d commit 7b3e87a64f8d416bd1266d6dc76efd1a69a94977 @jetheredge jetheredge committed Dec 15, 2011
Showing with 2 additions and 56 deletions.
  1. +1 −27 SquishIt.Framework/Base/BundleBase.cs
  2. +1 −29 SquishIt.Tests/CssBundleTests.cs
@@ -215,21 +215,7 @@ public T AddString(string format, params object[] values)
var content = string.Format(format, values);
return AddString(content);
}
-
- public T AddToGroup(string group, params string[] filesPath)
- {
- foreach (var filePath in filesPath)
- AddToGroup(group, filePath);
-
- return (T)this;
- }
-
- public T AddToGroup(string group, string filePath)
- {
- AddAsset(new Asset(filePath), group);
- return (T)this;
- }
-
+
public T AddRemote(string localPath, string remotePath)
{
return AddRemote(localPath, remotePath, false);
@@ -558,18 +544,6 @@ public T WithAttributes(Dictionary<string, string> attributes, bool merge = true
return (T)this;
}
- public T WithGroupAttribute(string name, string value, string group)
- {
- AddAttributes(new Dictionary<string, string> { { name, value } }, group);
- return (T)this;
- }
-
- public T WithGroupAttributes(Dictionary<string, string> attributes, string group, bool merge = true)
- {
- AddAttributes(attributes, group, merge);
- return (T)this;
- }
-
public T WithMinifier<TMin>() where TMin : IMinifier<T>
{
Minifier = MinifierFactory.Get<T, TMin>();
@@ -92,35 +92,7 @@ public void CanAddMultiplePathFiles()
Assert.AreEqual(assetBundle1.Order, assetBundle2.Order);
}
}
-
- [Test]
- public void CanAddMultiplePathFilesToGroup()
- {
- var myGroup = "myGroup";
- var cssBundle1 = cssBundleFactory
- .WithDebuggingEnabled(true)
- .Create();
-
- var cssBundle2 = cssBundleFactory
- .WithDebuggingEnabled(true)
- .Create();
-
- cssBundle1.AddToGroup(myGroup, "/css/first.css", "/css/second.css");
- cssBundle2.AddToGroup(myGroup, "/css/first.css").AddToGroup(myGroup, "/css/second.css");
-
- var cssBundle1Assets = cssBundle1.GroupBundles[myGroup].Assets;
- var cssBundle2Assets = cssBundle1.GroupBundles[myGroup].Assets;
-
- Assert.AreEqual(cssBundle1Assets.Count, cssBundle2Assets.Count);
- for (var i = 0; i < cssBundle1Assets.Count; i++)
- {
- var assetBundle1 = cssBundle1Assets[i];
- var assetBundle2 = cssBundle2Assets[i];
- Assert.AreEqual(assetBundle1.LocalPath, assetBundle2.LocalPath);
- Assert.AreEqual(assetBundle1.Order, assetBundle2.Order);
- }
- }
-
+
[Test]
public void CanBundleCss()
{

0 comments on commit 7b3e87a

Please sign in to comment.