exportRules() throws ⚡️ `Cannot read property '_jsonRequest' of undefined`

[adrienjoly]

Do you want to request a feature or report a bug?

bug

What is the current behavior?

After loading https://cdn.jsdelivr.net/algoliasearch/3/algoliasearch.min.js (v3.24.9), the JavaScript code below throws TypeError: Cannot read property '_jsonRequest' of undefined

const fromClient = algoliasearch(fromAppId, fromApiKey);
const fromIndex = fromClient.initIndex(fromIdxName);
fromIndex.exportRules(100, console.log);

Tested on both nodejs (back-end) and in jsfiddle (front-end) with the same version.

I relied on the following documentation: https://www.algolia.com/doc/api-reference/api-methods/rules-export/?language=javascript#examples

What is the expected behavior?

I expected to have a list query rules printed to my javascript console.

[Haroenv]

Whoow that’s an error. I think it’s because the context is wrong, we might need to bind something there. I’ll take a look 👀

[adrienjoly]

What's up @Haroenv ?

I may need to use that function soon. => Is there anything I can do to help / speed up the solving of this issue?

[vvo]

Yes everytime we do this

algoliasearch-client-javascript/src/Index.js

Lines 801 to 803 in 58a4b2d

	Index.prototype.exportRules = function(hitsPerPage, callback) {
	return exportData(this.searchRules, hitsPerPage, callback);
	};

It should be exportData(this.searchRules.bind(this)).
But there are multiple calls like that, were those tested somehow?

[Haroenv]

@vvo, they are tested in the integration tests, exportRules and exportSynonyms were both made in the same manner, a PR with maybe an even more integration test would be great

[vvo]

Here's an online test case: https://codesandbox.io/s/93lr68qyq4

This is broken for now.

Even if there seems to be integration tests, it might works only because either those ones are not run OR because we somehow bind the context in a forced way via bindAll? :)

Maybe add some unit test on top of that.