fix(data): set loaded to true on queryManySuccess (#3228)

Closes #3165 

BREAKING CHANGES:

Now both the `getWithQuery` and `getAll` methods are consistent and do set `loaded` property to true on dispatching their success actions respectively.

BEFORE:

The `getWithQuery` method would not set the `loaded` property to true upon success

AFTER:

The `getWithQuery` method sets the `loaded` property to true upon success

Author: WojciechKasprzyk

modules/data/spec/reducers/entity-collection-reducer.spec.ts

@@ -356,7 +356,7 @@ describe('EntityCollectionReducer', () => {
       const collection = state['Hero'];
 
       expect(collection.ids).toEqual([3]);
-      expect(collection.loaded).toBe(false);
+      expect(collection.loaded).toBe(true);
       expect(collection.loading).toBe(false);
     });
 

modules/data/src/reducers/entity-collection-reducer-methods.ts

@@ -290,6 +290,7 @@ export class EntityCollectionReducerMethods<T> {
         collection,
         mergeStrategy
       ),
+      loaded: true,
       loading: false,
     };
   }