Yasuke/moto
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Glue: get_table() should return latest version (#6006)

Browse Source
This commit is contained in:
Bert Blommers 2023-03-03 18:43:44 -01:00 committed by GitHub
parent a1a43e3f74
commit 96b8e12d45
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
moto/glue/models.py
View File

@ -840,7 +840,8 @@ class FakeTable(BaseModel):
def delete_version(self, version_id):
self.versions.pop(version_id)
def as_dict(self, version=1):
def as_dict(self, version=None):
version = version or self._current_version
obj = {
"DatabaseName": self.database_name,
"Name": self.name,

14
tests/test_glue/test_datacatalog.py
View File

@ -319,6 +319,11 @@ def test_get_table_versions():
helpers.create_table(client, database_name, table_name, table_input)
version_inputs["1"] = table_input
# Get table should retrieve the first version
table = client.get_table(DatabaseName=database_name, Name=table_name)["Table"]
table["StorageDescriptor"]["Columns"].should.equal([])
table["VersionId"].should.equal("1")
columns = [{"Name": "country", "Type": "string"}]
table_input = helpers.create_table_input(database_name, table_name, columns=columns)
helpers.update_table(client, database_name, table_name, table_input)
@ -354,6 +359,15 @@ def test_get_table_versions():
ver["Table"]["Name"].should.equal(table_name)
ver["Table"]["StorageDescriptor"]["Columns"].should.equal(columns)
# get_table should retrieve the latest version
table = client.get_table(DatabaseName=database_name, Name=table_name)["Table"]
table["StorageDescriptor"]["Columns"].should.equal(columns)
table["VersionId"].should.equal("3")
table = client.get_tables(DatabaseName=database_name)["TableList"][0]
table["StorageDescriptor"]["Columns"].should.equal(columns)
table["VersionId"].should.equal("3")
@mock_glue
def test_get_table_version_not_found():