[BUG] Cache does not seem to be working on FileList component

[russgove]

I am using the filelist component in a teams tab. The component keeps making calls to the graph to get the files even though they were already loaded and are in the cache.

 <FileList
            siteId={props.displayFolder.SiteId}
            driveId={props.displayFolder.DriveId}
            itemId={localFolderId}
            disableOpenOnClick={true}
            pageSize={10}

            key={localFolderId}
            itemClick={itemSelected()}>
            <MyEvent template="loading"></MyEvent>
        </FileList>

To Reproduce
Steps to reproduce the behavior:

1. Add a filelist component to a teams tab.
2. Run the app
3. Start fiddler
4. reload the page several times
5. You will see the calls made to the graph to get the files on each reload

Expected behavior
The files should be retrieved from the cache
n your problem.

Environment (please complete the following information):

• OS: Windows
• Browser Edge
• Framework react
• ContextMicrosoft Teams
• Version 4.2.3
• Provider ProxyProvider

Additional context
In troubleshooting this i ended up looking at the GetIterator method in graph.files.ts
When we are getting files from the cache we use this:
const fileList = await getFileListFromCache(cache, storeName, ${endpoint}:${top});

When adding files to the cache we use this

await cache.putValue(endpoint, {
files: filesPageIterator.value.map(v => JSON.stringify(v)),
nextLink
});

so the keys to not seem to match. When we get the key is ${endpoint}:${top}, when we put the key is endpoint.

[musale]

@russgove thank you for figuring this out! I can confirm that the keys are inconsistent in

microsoft-graph-toolkit/packages/mgt-components/src/graph/graph.files.ts

Line 257 in 62b1ed4

const fileList = await getFileListFromCache(cache, storeName, `${endpoint}:${top}`);

and in getting the values here:

microsoft-graph-toolkit/packages/mgt-components/src/graph/graph.files.ts

Lines 275 to 278 in 62b1ed4

	await cache.putValue(endpoint, {
	files: filesPageIterator.value.map(v => JSON.stringify(v)),
	nextLink
	});

The key can be moved out to its own variable and reused throughout the code block. Would you be interested in putting up a PR for this?