Navigation Menu

Skip to content

Commit

Permalink
isSynonymsTableAvailableSync => hasSynonymsTableSync
Browse files Browse the repository at this point in the history
  • Loading branch information
piroor committed Aug 9, 2012
1 parent 6ade8e0 commit e0e6553
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion lib/api/2011-02-01/search.js
Expand Up @@ -125,7 +125,7 @@ exports.createHandler = function(context) {
match_columns: matchColumns
};

if (domain.isSynonymsTableAvailableSync()) {
if (domain.hasSynonymsTableSync()) {
options.query_expansion = domain.synonymsTableName + '.synonyms';
}
if (filter) {
Expand Down
6 changes: 3 additions & 3 deletions lib/database/domain.js
Expand Up @@ -256,7 +256,7 @@ Domain.prototype = {
if (!this.context)
throw new Error('no context');

if (!this.isSynonymsTableAvailableSync())
if (!this.hasSynonymsTableSync())
return {};

var options = {
Expand Down Expand Up @@ -292,7 +292,7 @@ Domain.prototype = {
},

setupBlankSynonymsTable: function() {
if (this.isSynonymsTableAvailableSync()) {
if (this.hasSynonymsTableSync()) {
this.context.commandSync('table_remove', {
table: this.synonymsTableName
});
Expand All @@ -311,7 +311,7 @@ Domain.prototype = {
});
},

isSynonymsTableAvailableSync: function() {
hasSynonymsTableSync: function() {
var tables = this.context.tableListSync();
return tables.some(function(table) {
return table.name === this.synonymsTableName;
Expand Down
4 changes: 2 additions & 2 deletions test/api-configuration.test.js
Expand Up @@ -777,14 +777,14 @@ suite('Configuration API', function() {
})
.next(function() {
domain = new Domain('companies', context);
assert.isFalse(domain.isSynonymsTableAvailableSync());
assert.isFalse(domain.hasSynonymsTableSync());
})
.get('/?Version=2011-02-01&Action=UpdateSynonymOptions&' +
'DomainName=companies&Synonyms='+synonyms, {
'Host': 'cloudsearch.localhost'
})
.next(function(response) {
assert.isTrue(domain.isSynonymsTableAvailableSync());
assert.isTrue(domain.hasSynonymsTableSync());

response = toParsedResponse(response);
assert.deepEqual(response.pattern,
Expand Down
4 changes: 2 additions & 2 deletions test/database-domain.test.js
Expand Up @@ -304,13 +304,13 @@ suite('database', function() {

test('updateSynonymsSync', function() {
var domain = new Domain('companies', context);
assert.isFalse(domain.isSynonymsTableAvailableSync());
assert.isFalse(domain.hasSynonymsTableSync());

domain.updateSynonymsSync({
tokio: ['tokyo'],
dekkaido: 'hokkaido'
});
assert.isTrue(domain.isSynonymsTableAvailableSync());
assert.isTrue(domain.hasSynonymsTableSync());

var dumpExpected =
'table_create ' + domain.synonymsTableName + ' ' +
Expand Down

0 comments on commit e0e6553

Please sign in to comment.