Merge pull request #178 from Nfactor26/ignore-deleted-fixtures-while-retrieving

Nfactor26 · web-flow · commit d90ec1a23ea5 · 2022-11-15T21:06:52.000+05:30
Exclude deleted fixtures when retrieving fixtures for a given version of project
diff --git a/src/Pixel.Persistence.Respository/TestFixtureRepository.cs b/src/Pixel.Persistence.Respository/TestFixtureRepository.cs
@@ -57,8 +57,8 @@ public async Task<TestFixture> FindByNameAsync(string projectId, string projectV
     /// <inheritdoc/>  
     public async Task<IEnumerable<TestFixture>> GetFixturesAsync(string projectId, string projectVersion, CancellationToken cancellationToken)
     {
-        var filter = Builders<TestFixture>.Filter.And(Builders<TestFixture>.Filter.Eq(x => x.ProjectId, projectId),
-            Builders<TestFixture>.Filter.Eq(x => x.ProjectVersion, projectVersion));
+        var filter = Builders<TestFixture>.Filter.Eq(x => x.ProjectId, projectId) & Builders<TestFixture>.Filter.Eq(x => x.ProjectVersion, projectVersion) &
+             Builders<TestFixture>.Filter.Eq(x => x.IsDeleted, false);
         var fixtures = await fixturesCollection.FindAsync(filter, FindOptions, cancellationToken);
         return await fixtures.ToListAsync();       
     }

Original file line number	Diff line number	Diff line change
`@@ -57,8 +57,8 @@ public async Task<TestFixture> FindByNameAsync(string projectId, string projectV`
`57`	`57`	`/// <inheritdoc/>`
`58`	`58`	`public async Task<IEnumerable<TestFixture>> GetFixturesAsync(string projectId, string projectVersion, CancellationToken cancellationToken)`
`59`	`59`	`{`
`60`		`- var filter = Builders<TestFixture>.Filter.And(Builders<TestFixture>.Filter.Eq(x => x.ProjectId, projectId),`
`61`		`- Builders<TestFixture>.Filter.Eq(x => x.ProjectVersion, projectVersion));`
	`60`	`+ var filter = Builders<TestFixture>.Filter.Eq(x => x.ProjectId, projectId) & Builders<TestFixture>.Filter.Eq(x => x.ProjectVersion, projectVersion) &`
	`61`	`+ Builders<TestFixture>.Filter.Eq(x => x.IsDeleted, false);`
`62`	`62`	`var fixtures = await fixturesCollection.FindAsync(filter, FindOptions, cancellationToken);`
`63`	`63`	`return await fixtures.ToListAsync();`
`64`	`64`	`}`