Пара вопросов.

[dim5x]

Саш, по пункту 1:
Можно же брать напрямую из атрибута функции, без перекладывания в новую переменную.
По пункту 2:
Получается, что db.execute_non_query() - используется до объявления. Потому что экземпляр класса создаётся в ветке if, а туда может и не занести выполнение программы. Но это ладно - работает и ок :)
По пункту 3:
Какое-то странное разделение if получилось.

Может быть нужно как-то так:

def insert_data(data, table, conn='not_created'):
    """ Добавляем данные в базу. """
    columns, values = '', ''
    for key in data:
        if columns == '':
            columns += key
            values += get_value(data[key])
        else:
            columns = columns + ', ' + key
            values = values + ', ' + get_value(data[key])

    query = 'insert into ' + table + '(' + columns + ') values(' + values + ')'

    if conn == 'not_created':
        db = DatabaseConnection()
        db.open()
        db.execute_non_query(query)
        db.close()
    else:
        db.execute_non_query(query)
        db.close()