fix($resource): ignore undefined parameters

- $resource should handle multiple params with same name
- ignore slashes of undefined parameters
- fix default parameters issue, mentioned in #875

Closes #875
Closes #782
This commit is contained in:
petrovalex
2012-05-11 23:14:59 +03:00
committed by Misko Hevery
parent a56aaa9877
commit d2e52b2376
2 changed files with 35 additions and 6 deletions

View File

@@ -55,12 +55,14 @@ describe("resource", function() {
$httpBackend.expect('GET', '/Path');
$httpBackend.expect('GET', '/Path/1');
$httpBackend.expect('GET', '/Path/2/3');
$httpBackend.expect('GET', '/Path/4/5/6');
$httpBackend.expect('GET', '/Path/4/5');
$httpBackend.expect('GET', '/Path/6/7/8');
R.get({});
R.get({a:1});
R.get({a:2, b:3});
R.get({a:4, b:5, c:6});
R.get({a:4, c:5});
R.get({a:6, b:7, c:8});
});
@@ -123,6 +125,27 @@ describe("resource", function() {
});
it('should build resource with action default param reading the value from instance', function() {
$httpBackend.expect('POST', '/Customer/123').respond();
var R = $resource('/Customer/:id', {}, {post: {method: 'POST', params: {id: '@id'}}});
var inst = new R({id:123});
expect(inst.id).toBe(123);
inst.$post();
});
it('should handle multiple params with same name', function() {
var R = $resource('/:id/:id');
$httpBackend.when('GET').respond('{}');
$httpBackend.expect('GET', '/1/1');
R.get({id:1});
});
it("should create resource", function() {
$httpBackend.expect('POST', '/CreditCard', '{"name":"misko"}').respond({id: 123, name: 'misko'});